fix(server): hostname defaults to localhost, fix #3355

[patak-dev]

Description

Rework of #3382, avoiding replacing '127.0.0.1' by 'localhost' if this was defined by the user using host

It also fixes options.host === false.

The hint to use host in the console is not shown if the user sets 127.0.0.1 by hand.

---

Fix #3355

After #2977, we lost the hostname defaulting to localhost for '127.0.0.1' and '0.0.0.0'

vite/packages/vite/src/node/server/index.ts

Line 536 in c374a54

if (hostname === '0.0.0.0') hostname = 'localhost'

This PR adds that logic back, also including host === undefined and '::' that should be treated equally to '0.0.0.0'

The reported URL in the console also replaces '127.0.0.1' by localhost now.

---

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.

--

Replaces + closes #3382

[nihalgonsalves]

I like this structure 👍🏼 It's more understandable now, and there aren't replacements in different places.