Skip to content

fix: Use current host URL instead of hardcoded localhost for API calls#7

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

fix: Use current host URL instead of hardcoded localhost for API calls#7
StuartF303 merged 1 commit intomainfrom
claude/fly-io-deployment-011CUpsFKnx2JFyoEDjhfgbz

Conversation

@StuartF303
Copy link
Copy Markdown
Owner

Changed frontend to use dynamic base URL instead of hardcoded localhost:5000.

Problem:

Solution:

  • Use builder.HostEnvironment.BaseAddress for API base URL
  • This automatically uses the current host:
  • Nginx proxies /api and /hubs requests to the backend API (port 5000)

Result:

Changed frontend to use dynamic base URL instead of hardcoded localhost:5000.

Problem:
- Frontend was hardcoded to use http://localhost:5000 for API calls
- In production on Fly.io, this caused requests to fail (localhost in browser)
- Browser tried accessing http://localhost:5000/api/tanks instead of the deployed URL

Solution:
- Use builder.HostEnvironment.BaseAddress for API base URL
- This automatically uses the current host:
  - Production (Fly.io): https://vandaemon.fly.dev
  - Local dev: http://localhost:8080 (or whatever port serves the app)
- Nginx proxies /api and /hubs requests to the backend API (port 5000)

Result:
- API calls now work correctly: https://vandaemon.fly.dev/api/tanks
- SignalR connects to: https://vandaemon.fly.dev/hubs/telemetry
- Works in both production and local development environments
@StuartF303 StuartF303 merged commit 11b79fa 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