Skip to content

fix: Replace all hard-coded port 6989 with SERVER_PORT env var#1225

Open
eduardomozart wants to merge 3 commits into
gnmyt:mainfrom
eduardomozart:copilot/update-port-references-to-env
Open

fix: Replace all hard-coded port 6989 with SERVER_PORT env var#1225
eduardomozart wants to merge 3 commits into
gnmyt:mainfrom
eduardomozart:copilot/update-port-references-to-env

Conversation

@eduardomozart
Copy link
Copy Markdown
Contributor

@eduardomozart eduardomozart commented Mar 3, 2026

📋 Description

Hard-coded port 6989 was scattered across source and config files, preventing SERVER_PORT from being respected end-to-end.

Changes

  • server/index.js — already used process.env.SERVER_PORT || 6989 (no change needed)
  • client/vite.config.js — dev proxy target now derives port from SERVER_PORT:
    target: `http://localhost:${process.env.SERVER_PORT || 6989}`
  • server/openapi.js — development API server URL in the OpenAPI spec:
    { url: `http://localhost:${process.env.SERVER_PORT || 6989}/api`, ... }
  • landing/src/pages/Install/Install.jsx — access URL hint uses Vite's build-time env convention:
    <code>http://localhost:{import.meta.env.VITE_SERVER_PORT || 6989}</code>
  • Dockerfile — retains EXPOSE 6989 as a static documentation hint; actual runtime port is controlled via SERVER_PORT

🚀 Changes made to ...

  • 🔧 Server
  • 🖥️ Client
  • 📚 Documentation
  • 🔄 Other: ___

✅ Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have looked for similar pull requests in the repository and found none
  • This pull request does not contain translations (translations are managed through Crowdin)

🔗 Related Issues

Copilot AI and others added 3 commits March 3, 2026 13:02
Co-authored-by: eduardomozart <2974895+eduardomozart@users.noreply.github.com>
…T env var

Co-authored-by: eduardomozart <2974895+eduardomozart@users.noreply.github.com>
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Mar 3, 2026

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