fix: respect new port when change the config file

[CHOYSEN]

Description

Fixes #6074

What is the purpose of this pull request?

• Bug fix
• New Feature
• Documentation update
• Other

Before submitting the PR, please make sure you do the following

• Read the Contributing Guidelines.
• Read the Pull Request Guidelines and follow the Commit Convention.
• Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
• Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
• Ideally, include relevant tests that fail without this PR but pass with it.

[patak-dev]

@CHOYSEN did you test what happens when you have two dev servers running (port 3000 and 3001) and change the port?

[CHOYSEN]

It works as expect, i try to change the first dev server from 3000 to 3001. @patak-dev

test@C02FV5VTMD6M vite-config-issue 2 % pnpm dev

> [email protected] dev /Users/test/Desktop/vite-config-issue 2
> vite


  vite v2.7.1 dev server running at:

  > Local: http://localhost:3000/
  > Network: use `--host` to expose

  ready in 403ms.

下午5:03:26 [vite] vite.config.ts changed, restarting server...
Port 3001 is in use, trying another one...
下午5:03:26 [vite] server restarted.


  > Local: http://localhost:3002/
  > Network: use `--host` to expose

[CHOYSEN]

For strictPort, we print the error and waiting for user to change the port again.

import { defineConfig } from "vite";

export default defineConfig({
  server: {
    port: 3000,
    strictPort: true
  }
});

Change to port 3001:

test@C02FV5VTMD6M vite-config-issue 2 % yarn dev
yarn run v1.22.10
warning package.json: No license field
$ vite

  vite v2.7.1 dev server running at:

  > Local: http://localhost:3000/
  > Network: use `--host` to expose

  ready in 380ms.

上午11:30:31 [vite] vite.config.ts changed, restarting server...
Error: Port 3001 is already in use

Change to port 3002:

上午11:31:07 [vite] vite.config.ts changed, restarting server...
上午11:31:07 [vite] server restarted.


  > Local: http://localhost:3002/
  > Network: use `--host` to expose

[bluwy]

It would be nice to handle host as well so we can close #5996.

[CHOYSEN]

It would be nice to handle host as well so we can close #5996.

Hi @btea, would you like to agree this?

[btea]

It would be nice to handle host as well so we can close #5996.

Hi @btea, would you like to agree this?

Yes.

[CHOYSEN]

It would be nice to handle host as well so we can close #5996.

Hi @btea, would you like to agree this?

Yes.

Thanks!

[bluwy]

Not sure what's up with CI, but this looks good to me.

[patak-dev]

I re-run the jobs, CI is a bit moody lately. We need to move away from Jest, we are working on it 😉