Skip to content

Fix Fly.io deployment timeout issues#2

Merged
bokelley merged 1 commit intomainfrom
slow-deploy
Sep 4, 2025
Merged

Fix Fly.io deployment timeout issues#2
bokelley merged 1 commit intomainfrom
slow-deploy

Conversation

@bokelley
Copy link
Copy Markdown
Contributor

@bokelley bokelley commented Sep 4, 2025

Summary

  • Fix Fly.io deployment hanging on health checks
  • Replace TCP health checks with HTTP health checks pointing to /health endpoint
  • Update Dockerfile to properly build TypeScript and use correct production files
  • Fix port consistency (8080) across all configuration files

Changes Made

  • fly.toml: Changed from TCP checks to HTTP health checks at /health
  • Dockerfile: Added TypeScript build step and proper file copying for production
  • server.js: Fixed default port from 3000 to 8080 for consistency

Test Plan

  • Verify health check endpoint is accessible
  • Deploy to Fly.io and confirm deployment succeeds without timeout
  • Test application functionality after deployment

🤖 Generated with Claude Code

- Replace TCP health checks with HTTP health checks pointing to /health endpoint
- Update Dockerfile to properly build TypeScript and use correct server files
- Fix default port consistency (8080) across all configuration files
- Use npm start command for proper production deployment

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@bokelley bokelley merged commit 31cc013 into main Sep 4, 2025
1 check passed
tescoboy pushed a commit to tescoboy/adcp-client that referenced this pull request Apr 14, 2026
…tprotocol#3) in Common Mistakes table

Removes traps adcontextprotocol#1, adcontextprotocol#4, adcontextprotocol#5, adcontextprotocol#6 per reviewer feedback:

adcontextprotocol#1 and adcontextprotocol#6 have accuracy issues
adcontextprotocol#4 and adcontextprotocol#5 to be filed as GitHub Issues (may be Addie-specific)
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