Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(web): provide free port to http-server #16917

Merged

Conversation

barbados-clemens
Copy link
Contributor

@barbados-clemens barbados-clemens commented May 10, 2023

Current Behavior

@nx/web:file-server doesn't return the port used by http-server if the port is already in use.

Expected Behavior

@nx/web:file-server returns the port value used by http-server by providing a known free port.

Related Issue(s)

Fixes #16684

@vercel
Copy link

vercel bot commented May 10, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
nx-dev ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 19, 2023 3:39pm

@barbados-clemens barbados-clemens self-assigned this May 10, 2023
@barbados-clemens barbados-clemens added the scope: testing tools Issues related to Cypress / Jest / Playwright / Vitest support in Nx label May 10, 2023
@barbados-clemens barbados-clemens force-pushed the fix/web-file-server-port branch from 2a45ed1 to bded998 Compare May 19, 2023 15:34
@barbados-clemens barbados-clemens changed the title fix(web): return port from http-server process fix(web): provide free port to http-server May 19, 2023
@barbados-clemens barbados-clemens marked this pull request as ready for review May 19, 2023 16:13
@barbados-clemens barbados-clemens merged commit 4896126 into nrwl:master May 19, 2023
@barbados-clemens barbados-clemens deleted the fix/web-file-server-port branch May 19, 2023 16:55
@github-actions
Copy link

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
scope: testing tools Issues related to Cypress / Jest / Playwright / Vitest support in Nx
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cannot use cypress port 0 to auto-assign a port if the devServerTarget is @nrwl/web:file-server
2 participants