fix(dts-plugin): hold the broker server if the remote not start local…

…ly (#3202)
module-federation · Nov 11, 2024 · 85990e2 · 85990e2
1 parent 1a80706
commit 85990e2
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 2 deletions.
diff --git a/.changeset/fair-plums-live.md b/.changeset/fair-plums-live.md
@@ -0,0 +1,5 @@
+---
+'@module-federation/dts-plugin': patch
+---
+
+fix(dts-plugin): hold the broker server if the remote not start locally
diff --git a/packages/dts-plugin/src/server/DevServer.ts b/packages/dts-plugin/src/server/DevServer.ts
@@ -314,8 +314,19 @@ export class ModuleFederationDevServer {
         'warn',
       );
 
-      this._subscriberWebsocketMap[identifier] &&
-        this._subscriberWebsocketMap[identifier].close();
+      this._subscriberWebsocketMap[identifier]?.close();
+      delete this._subscriberWebsocketMap[identifier];
+    });
+
+    this._subscriberWebsocketMap[identifier].on('error', (event) => {
+      if ('code' in event && event.code === 'ETIMEDOUT') {
+        fileLog(
+          `Can not connect ${JSON.stringify(remote)}, please make sure this remote is started locally.`,
+          MF_SERVER_IDENTIFIER,
+          'warn',
+        );
+      }
+      this._subscriberWebsocketMap[identifier]?.close();
       delete this._subscriberWebsocketMap[identifier];
     });
   }