fix(dev): avoid event emitter leak caused by server.listen callback (#21451)

Author: sapphi-red

packages/vite/src/node/__tests__/plugins/hooks.spec.ts

@@ -57,9 +57,9 @@ const createPreviewServerWithPlugin = async (plugin: Plugin) => {
         name: 'mock-preview',
         configurePreviewServer({ httpServer }) {
           // NOTE: make httpServer.listen no-op to avoid starting a server
-          httpServer.listen = (...args: unknown[]) => {
-            const listener = args.at(-1) as () => void
-            listener()
+          httpServer.listen = () => {
+            const lastListener = httpServer.listeners('listening').at(-1)!
+            lastListener.call(httpServer)
             return httpServer as any
           }
         },

packages/vite/src/node/http.ts

@@ -197,14 +197,20 @@ async function tryBindServer(
 > {
   return new Promise((resolve) => {
     const onError = (e: Error & { code?: string }) => {
-      httpServer.removeListener('error', onError)
+      httpServer.off('error', onError)
+      httpServer.off('listening', onListening)
       resolve({ success: false, error: e })
     }
-    httpServer.on('error', onError)
-    httpServer.listen(port, host, () => {
-      httpServer.removeListener('error', onError)
+    const onListening = () => {
+      httpServer.off('error', onError)
+      httpServer.off('listening', onListening)
       resolve({ success: true })
-    })
+    }
+
+    httpServer.on('error', onError)
+    httpServer.on('listening', onListening)
+
+    httpServer.listen(port, host)
   })
 }