fix: move server start before url resolve

vitejs · Jan 10, 2023 · aab2019 · aab2019
1 parent 1a802f7
commit aab2019
Showing 1 changed file with 12 additions and 11 deletions.
diff --git a/packages/vite/src/node/preview.ts b/packages/vite/src/node/preview.ts
@@ -93,13 +93,25 @@ export async function preview(
   const logger = config.logger
 
   const app = connect() as Connect.Server
+
+  const hostname = await resolveHostname(options.host)
+  const port = options.port ?? 4173
+  const protocol = options.https ? 'https' : 'http'
+
   const httpServer = await resolveHttpServer(
     config.preview,
     app,
     await resolveHttpsConfig(config.preview?.https),
   )
   setClientErrorHandler(httpServer, config.logger)
 
+  const serverPort = await httpServerStart(httpServer, {
+    port,
+    strictPort: options.strictPort,
+    host: hostname.host,
+    logger,
+  })
+
   const resolvedUrls = await resolveServerUrls(
     httpServer,
     config.preview,
@@ -162,17 +174,6 @@ export async function preview(
   // apply post server hooks from plugins
   postHooks.forEach((fn) => fn && fn())
 
-  const hostname = await resolveHostname(options.host)
-  const port = options.port ?? 4173
-  const protocol = options.https ? 'https' : 'http'
-
-  const serverPort = await httpServerStart(httpServer, {
-    port,
-    strictPort: options.strictPort,
-    host: hostname.host,
-    logger,
-  })
-
   if (options.open) {
     const path = typeof options.open === 'string' ? options.open : previewBase
     openBrowser(