Skip to content

Conversation

@cte
Copy link
Collaborator

@cte cte commented Jun 6, 2025

Description

Some people have conflicts on port 3000.


Important

Change evals web app port from 3000 to 8080 to avoid conflicts, updating scripts, documentation, and Docker configurations accordingly.

  • Behavior:
    • Change evals web app port from 3000 to 8080 in package.json, docker-compose.yml, and setup.sh.
    • Update README.md to reflect new port 8080 for web service access.
  • Scripts:
    • Modify dev script in package.json to use --port 8080.
    • Update port check in setup.sh to 8080.
  • Docker:
    • Change port mapping in docker-compose.yml from 3000:3000 to 8080:3000.

This description was created by Ellipsis for 618af52. You can customize this summary. It will automatically update as commits are pushed.

@cte cte requested review from jr and mrubens as code owners June 6, 2025 16:16
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jun 6, 2025
@dosubot dosubot bot added lgtm This PR has been approved by a maintainer size:S This PR changes 10-29 lines, ignoring generated files. and removed size:XS This PR changes 0-9 lines, ignoring generated files. labels Jun 6, 2025
@adamhill
Copy link
Contributor

adamhill commented Jun 6, 2025

Just sayin... 8080 is used even more often by other things (HTTP proxies, sniffers, web tools etc...) 8081 too. I dunno how to pick a non used port nowdays.

Thanks Vint Cerf :-)

@daniel-lxs daniel-lxs moved this from Triage to PR [Needs Prelim Review] in Roo Code Roadmap Jun 7, 2025
@daniel-lxs daniel-lxs moved this from PR [Needs Prelim Review] to PR [Draft / In Progress] in Roo Code Roadmap Jun 7, 2025
@cte
Copy link
Collaborator Author

cte commented Jun 9, 2025

Will re-visit this one; I think we also want non-standard ports for postgres and redis.

@cte cte closed this Jun 9, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jun 9, 2025
@github-project-automation github-project-automation bot moved this from PR [Draft / In Progress] to Done in Roo Code Roadmap Jun 9, 2025
@cte cte deleted the cte/evals-port-8080 branch June 9, 2025 21:27
@jr
Copy link
Collaborator

jr commented Jun 10, 2025

@cte for postgres and redis you could also just use expose instead of port (allow the linked services to access them, but not expose on the host), unless you're regularly poking around in them.

@jr jr mentioned this pull request Jun 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer PR - Draft / In Progress size:S This PR changes 10-29 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

6 participants