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: better port inuse detection #16641

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

CGQAQ
Copy link
Contributor

@CGQAQ CGQAQ commented May 9, 2024

Description

If you have lets say "localhost:5173" is taken by another process, but you set the host to 0.0.0.0 and listen to the same port, vite will not be able to detect the port is in use, and localhost:5173 is not actually listened by vite as of now.
This PR addressed that by check every single address with the port to detect any of the address with the port was taken

follow up of #10651
Fixed #10638

Copy link

stackblitz bot commented May 9, 2024

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

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.

Check 0.0.0.0 as well as localhost/127.0.0.1 when checking for used port
1 participant