Skip to content

fix: api port and routes#113

Merged
bniladridas merged 1 commit into
mainfrom
fix/api-port-and-routes
Apr 19, 2026
Merged

fix: api port and routes#113
bniladridas merged 1 commit into
mainfrom
fix/api-port-and-routes

Conversation

@bniladridas

Copy link
Copy Markdown
Contributor

This PR fixes the default API server port to 5001 to avoid conflicts with macOS AirPlay Receiver, adds a root endpoint for API discovery, supports /health without version prefix, updates README examples, and corrects environment variable types.

Changes:

  • Update default port from 5000 to 5001 in api/server.py
  • Add root '/' endpoint returning API info
  • Support /health endpoint for both /health and /v1/health
  • Update README.md examples to use port 5001
  • Fix PORT env var default to string type

Use string '5001' instead of int 5001 for os.getenv default parameter.
@changeset-bot

changeset-bot Bot commented Apr 19, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 3d4dd5f

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions

Copy link
Copy Markdown
Contributor

☑️ I checked the pre-commit hooks and there was nothing to fix at commit 3d4dd5f.

@bniladridas
bniladridas merged commit 4507f76 into main Apr 19, 2026
36 of 37 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 19, 2026
@github-actions
github-actions Bot deleted the fix/api-port-and-routes branch April 19, 2026 14:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants