Merge pull request #105 from JJ-Cro/master

fix(v1.1.7): update circular ws issue

Author: tiagosiebler

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "coinbase-api",
-  "version": "1.1.6",
+  "version": "1.1.7",
   "description": "Node.js SDK for Coinbase's Advanced Trade, App, Exchange, International, Prime & Commerce REST APIs and WebSockets, with TypeScript & strong end to end tests.",
   "scripts": {
     "clean": "rm -rf dist",

package.json

@@ -375,12 +375,15 @@ export abstract class BaseWebsocketClient<
         return { wsKey };
       }
 
-      if (this.wsStore.isConnectionAttemptInProgress(wsKey)) {
+      if (
+        // Important: don't check for RECONNECTING here, or this clashes with reconnectWithDelay()!
+        this.wsStore.isConnectionState(wsKey, WsConnectionStateEnum.CONNECTING)
+      ) {
         this.logger.error(
           'Refused to connect to ws, connection attempt already active',
           { ...WS_LOGGER_CATEGORY, wsKey },
         );
-        return;
+        return this.wsStore.getConnectionInProgressPromise(wsKey)?.promise;
       }
 
       if (