Skip to content

fix: allow dynamic server port via environment variable#530

Open
giuliolibrando wants to merge 1 commit intoC4illin:mainfrom
giuliolibrando:patch-1
Open

fix: allow dynamic server port via environment variable#530
giuliolibrando wants to merge 1 commit intoC4illin:mainfrom
giuliolibrando:patch-1

Conversation

@giuliolibrando
Copy link

@giuliolibrando giuliolibrando commented Feb 2, 2026

Replaced hardcoded port 3000 with process.env.PORT in src/index.tsx to support custom deployment environments.


Summary by cubic

Server now reads the port from process.env.PORT, falling back to 3000. This enables deployments on platforms that assign dynamic ports.

Written for commit b08bcca. Summary will update on new commits.

Replaced hardcoded port 3000 with process.env.PORT in src/index.tsx to support custom deployment environments.
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

@C4illin
Copy link
Owner

C4illin commented Feb 3, 2026

When would you need this in a deployment?

@giuliolibrando
Copy link
Author

giuliolibrando commented Feb 3, 2026

Cause I used the proxmox script helper ( https://community-scripts.github.io/ProxmoxVE/scripts?id=convertx ) and I was not able to change the port to 80. It doesn't use docker, I would have had to expose the traffic via nginx proxy manager but this way less pieces are needed. Why hardcode the port?

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