fix(client): send complete message and close only if socket is still …

…open
ilijaNL · Aug 26, 2020 · 49b75ce · 49b75ce
1 parent 5c01ed9
commit 49b75ce
Showing 1 changed file with 11 additions and 7 deletions.
diff --git a/src/client.ts b/src/client.ts
@@ -232,19 +232,23 @@ export function createClient(options: ClientOptions): Client {
 
       return () => {
         if (socket) {
-          socket.send(
-            stringifyMessage<MessageType.Complete>({
-              id: uuid,
-              type: MessageType.Complete,
-            }),
-          );
+          if (socket.readyState === WebSocket.OPEN) {
+            socket.send(
+              stringifyMessage<MessageType.Complete>({
+                id: uuid,
+                type: MessageType.Complete,
+              }),
+            );
+          }
 
           socket.removeEventListener('message', handleMessage);
 
           // equal to 1 because this sink is the last one.
           // the deletion from the map happens afterwards
           if (Object.entries(subscribedSinks).length === 1) {
-            socket.close(1000, 'Normal Closure');
+            if (socket.readyState === WebSocket.OPEN) {
+              socket.close(1000, 'Normal Closure');
+            }
             socket = null;
           }
         }