fix: open first url if host does not match any urls (#19886)

Author: sapphi-red

packages/vite/src/node/__tests__/utils.spec.ts

@@ -12,6 +12,7 @@ import {
   generateCodeFrame,
   getHash,
   getLocalhostAddressIfDiffersFromDNS,
+  getServerUrlByHost,
   injectQuery,
   isFileReadable,
   mergeWithDefaults,
@@ -21,6 +22,7 @@ import {
   resolveHostname,
 } from '../utils'
 import { isWindows } from '../../shared/utils'
+import type { CommonServerOptions, ResolvedServerUrls } from '..'
 
 describe('bareImportRE', () => {
   test('should work with normal package name', () => {
@@ -725,3 +727,54 @@ describe('combineSourcemaps', () => {
     )
   })
 })
+
+describe('getServerUrlByHost', () => {
+  const urls: ResolvedServerUrls = {
+    local: ['http://localhost:5173'],
+    network: ['http://foo.example.com:5173'],
+  }
+  const cases = [
+    {
+      name: 'when host is undefined',
+      urls,
+      host: undefined,
+      expected: 'http://localhost:5173',
+    },
+    {
+      name: 'when host is true',
+      urls,
+      host: true,
+      expected: 'http://localhost:5173',
+    },
+    {
+      name: 'when host is explicit string',
+      urls,
+      host: 'foo.example.com',
+      expected: 'http://foo.example.com:5173',
+    },
+    {
+      name: 'when host is 0.0.0.0',
+      urls,
+      host: '0.0.0.0',
+      expected: 'http://localhost:5173',
+    },
+    {
+      name: 'when host is ::1',
+      urls,
+      host: '::1',
+      expected: 'http://localhost:5173',
+    },
+  ] satisfies ReadonlyArray<{
+    name: string
+    urls: ResolvedServerUrls
+    host: CommonServerOptions['host']
+    expected: string | undefined
+  }>
+
+  for (const { name, urls, host, expected } of cases) {
+    test(name, () => {
+      const actual = getServerUrlByHost(urls, host)
+      expect(actual).toBe(expected)
+    })
+  }
+})

packages/vite/src/node/preview.ts

@@ -277,7 +277,7 @@ export async function preview(
   )
 
   if (options.open) {
-    const url = getServerUrlByHost(server, options.host)
+    const url = getServerUrlByHost(server.resolvedUrls, options.host)
     if (url) {
       const path =
         typeof options.open === 'string' ? new URL(options.open, url).href : url

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

@@ -660,7 +660,7 @@ export async function _createServer(
     },
     openBrowser() {
       const options = server.config.server
-      const url = getServerUrlByHost(server, options.host)
+      const url = getServerUrlByHost(server.resolvedUrls, options.host)
       if (url) {
         const path =
           typeof options.open === 'string'

packages/vite/src/node/utils.ts

@@ -1658,15 +1658,17 @@ export const teardownSIGTERMListener = (
 }
 
 export function getServerUrlByHost(
-  server: PreviewServer | ViteDevServer,
+  resolvedUrls: ResolvedServerUrls | null,
   host: CommonServerOptions['host'],
 ): string | undefined {
-  if (typeof host !== 'string') {
-    return server.resolvedUrls?.local[0] ?? server.resolvedUrls?.network[0]
+  if (typeof host === 'string') {
+    const matchedUrl = [
+      ...(resolvedUrls?.local ?? []),
+      ...(resolvedUrls?.network ?? []),
+    ].find((url) => url.includes(host))
+    if (matchedUrl) {
+      return matchedUrl
+    }
   }
-
-  return [
-    ...(server.resolvedUrls?.local ?? []),
-    ...(server.resolvedUrls?.network ?? []),
-  ].find((url) => url.includes(host))
+  return resolvedUrls?.local[0] ?? resolvedUrls?.network[0]
 }