Skip to content

fix: Add /health endpoint for Fly.io health checks#6

Merged
StuartF303 merged 1 commit intomainfrom
claude/fly-io-deployment-011CUpsFKnx2JFyoEDjhfgbz
Nov 6, 2025
Merged

fix: Add /health endpoint for Fly.io health checks#6
StuartF303 merged 1 commit intomainfrom
claude/fly-io-deployment-011CUpsFKnx2JFyoEDjhfgbz

Conversation

@StuartF303
Copy link
Copy Markdown
Owner

Added health check endpoint to resolve 404 errors during deployment:

  • Added GET /health endpoint returning status and timestamp
  • Returns 200 OK with JSON response: {"status":"healthy","timestamp":"..."}

Also improved CORS configuration:

  • Development: Strict CORS with specific origins (localhost)
  • Production: Permissive CORS (traffic comes through nginx on same domain)

Fixes health check failures seen in Fly.io logs:
"Request reached the end of the middleware pipeline without being handled"

The health endpoint is proxied through nginx at /health on port 8080.

Added health check endpoint to resolve 404 errors during deployment:
- Added GET /health endpoint returning status and timestamp
- Returns 200 OK with JSON response: {"status":"healthy","timestamp":"..."}

Also improved CORS configuration:
- Development: Strict CORS with specific origins (localhost)
- Production: Permissive CORS (traffic comes through nginx on same domain)

Fixes health check failures seen in Fly.io logs:
  "Request reached the end of the middleware pipeline without being handled"

The health endpoint is proxied through nginx at /health on port 8080.
@StuartF303 StuartF303 merged commit 580e63b into main Nov 6, 2025
4 checks passed
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.

2 participants