Catch exception on failed WS command to avoid interface from breaking (…

…mattermost-community#3858) * Catch exception on failed WS command to avoid interface from breaking * Fix linter
kamre · Sep 16, 2022 · 68f03d3 · 68f03d3
1 parent 8ded583
commit 68f03d3
Showing 1 changed file with 10 additions and 6 deletions.
diff --git a/webapp/src/wsclient.ts b/webapp/src/wsclient.ts
@@ -172,13 +172,17 @@ class WSClient {
     }
 
     sendCommand(command: WSCommand): void {
-        if (this.client !== null) {
-            const {action, ...data} = command
-            this.client.sendMessage(this.clientPrefix + action, data)
-            return
-        }
+        try {
+            if (this.client !== null) {
+                const {action, ...data} = command
+                this.client.sendMessage(this.clientPrefix + action, data)
+                return
+            }
 
-        this.ws?.send(JSON.stringify(command))
+            this.ws?.send(JSON.stringify(command))
+        } catch (e) {
+            Utils.logError(`WSClient failed to send command ${command.action}: ${e}`)
+        }
     }
 
     sendAuthenticationCommand(token: string): void {