fix: call server.close() before closeAllConnections() to prevent new socket race

alex-xuweilong · alex-xuweilong · commit 80c074a12a83 · 2026-03-17T09:02:20.000+08:00
diff --git a/extensions/feishu/src/monitor.state.ts b/extensions/feishu/src/monitor.state.ts
@@ -141,11 +141,11 @@ export function closeHttpServer(server: http.Server): Promise<void> {
       resolve();
     }, SERVER_CLOSE_TIMEOUT_MS);
 
-    (server.closeAllConnections as (() => void) | undefined)?.();
     server.close(() => {
       clearTimeout(timer);
       resolve();
     });
+    (server.closeAllConnections as (() => void) | undefined)?.();
   });
 }
 
