Move bot health server off port 8080 to avoid Adminer conflict #130

Open
opened 2026-03-26 04:25:00 +00:00 by cal · 1 comment
Owner

Problem

The bot's health server (health_server.py) listens on port 8080, which is the same port Adminer is exposed on in the docker-compose stack. Curling localhost:8080/health from the host hits Adminer's login page instead of the bot's health endpoint.

Proposed Fix

Change the health server to a different port (e.g. 8081 or 9090) and update:

  • health_server.py — listen port
  • docker-compose.yml — exposed port mapping (if applicable)
  • Any monitoring/healthcheck configs that reference port 8080

Context

Discovered while verifying the bot was healthy after a deploy — curl -s http://localhost:8080/health returned Adminer HTML instead of the bot's health response.

## Problem The bot's health server (`health_server.py`) listens on port 8080, which is the same port Adminer is exposed on in the docker-compose stack. Curling `localhost:8080/health` from the host hits Adminer's login page instead of the bot's health endpoint. ## Proposed Fix Change the health server to a different port (e.g. 8081 or 9090) and update: - `health_server.py` — listen port - `docker-compose.yml` — exposed port mapping (if applicable) - Any monitoring/healthcheck configs that reference port 8080 ## Context Discovered while verifying the bot was healthy after a deploy — `curl -s http://localhost:8080/health` returned Adminer HTML instead of the bot's health response.
Claude added the
ai-working
label 2026-03-26 04:31:04 +00:00
Claude removed the
ai-working
label 2026-03-26 04:33:28 +00:00
Collaborator

PR #131 opened: #131

Changed health server default port from 8080 to 8081 in health_server.py. Updated all references in CLAUDE.md, tests/refractor-integration-test-plan.md, and tests/refractor-preflight.sh.

Note: docker-compose.yml is gitignored so can't be committed — the production compose on sba-bots will need its healthcheck URL updated to port 8081 manually when deploying.

PR #131 opened: https://git.manticorum.com/cal/paper-dynasty-discord/pulls/131 Changed health server default port from 8080 to 8081 in `health_server.py`. Updated all references in `CLAUDE.md`, `tests/refractor-integration-test-plan.md`, and `tests/refractor-preflight.sh`. **Note**: `docker-compose.yml` is gitignored so can't be committed — the production compose on `sba-bots` will need its healthcheck URL updated to port 8081 manually when deploying.
Claude added the
ai-pr-opened
label 2026-03-26 04:33:35 +00:00
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: cal/paper-dynasty-discord#130
No description provided.