Skip to content

fix(scripts): patch nginx proxy_pass for non-default CXDB HTTP port#52

Open
thewoolleyman wants to merge 1 commit intodanshapiro:mainfrom
thewoolleyman:fix/nginx-proxy-port-mismatch
Open

fix(scripts): patch nginx proxy_pass for non-default CXDB HTTP port#52
thewoolleyman wants to merge 1 commit intodanshapiro:mainfrom
thewoolleyman:fix/nginx-proxy-port-mismatch

Conversation

@thewoolleyman
Copy link
Copy Markdown
Contributor

Summary

  • The CXDB Docker image bakes nginx with proxy_pass targeting the default HTTP port (9010)
  • When KILROY_CXDB_HTTP_BASE_URL uses a different port, nginx returns 502 for all API requests, the in-container healthcheck fails, and the UI shows "Offline"
  • After container start, detect the port mismatch and patch nginx.conf before the health poll begins

Test plan

  • Start CXDB with non-default ports (e.g. KILROY_CXDB_HTTP_BASE_URL=http://127.0.0.1:9110) and verify the UI on the nginx port shows "Online"
  • Start CXDB with default ports and verify no patch is applied (idempotent)
  • Restart an existing container with non-default ports and verify nginx is re-patched

🤖 Generated with Claude Code

… port

The CXDB Docker image bakes nginx with proxy_pass targeting the default
HTTP port (9010). When KILROY_CXDB_HTTP_BASE_URL uses a different port,
nginx returns 502 for all API requests, the in-container healthcheck
fails, and the UI shows "Offline".

After container start, detect the mismatch and sed the correct port
into nginx.conf before the health poll begins.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@thewoolleyman thewoolleyman force-pushed the fix/nginx-proxy-port-mismatch branch from 5fcc55c to bf50b90 Compare March 22, 2026 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant