Improve error logging

Author: nielsandriesse

src/controllers/SessionController.ts

@@ -50,7 +50,7 @@ class SessionController implements SLWebSocketEventListener {
 
     initialize() {
         if (!this.authToken) {
-            console.log("Couldn't get session due to missing auth token.")
+            console.error("Couldn't get session due to missing auth token.")
             return
         }
         Server.getSession(this.authToken)
@@ -64,7 +64,7 @@ class SessionController implements SLWebSocketEventListener {
             .then(() => {
                 setTimeout(() => {
                     if (!this.authToken) {
-                        console.log("Couldn't connect web socket due to missing auth token.")
+                        console.error("Couldn't connect web socket due to missing auth token.")
                         return;
                     }
                     this.getWebSocket().slSend('connect-socket', this.authToken, {});

src/index.tsx

@@ -45,10 +45,10 @@ class Superluminal extends React.Component<Props, State> {
 
     setUser(user: { id: string, name: string }) {
         if (!user.id) {
-            return console.log('You must provide a valid user ID.');
+            return console.error('You must provide a valid user ID.');
         }
         if (!user.name) {
-            return console.log('You must provide a valid user name.');
+            return console.error('You must provide a valid user name.');
         }
         this.assistantViewRef.current?.setUser(user);
     }

src/networking/Server.ts

@@ -21,7 +21,7 @@ class Server {
             },
         })
         .then((response) => response.json())
-        .catch((error) => console.log(`Couldn't get session due to error: ${error?.reason ?? error}`));
+        .catch((error) => console.error(`Couldn't get session due to error: ${error?.reason ?? error}`));
     }
 }
 

src/networking/WebSocket.ts

@@ -48,6 +48,7 @@ export class SLWebSocket extends WebSocket {
 
     private static initializeWithBufferAndListeners(url: string, buffer: ArrayBuffer[], listeners: SLWebSocketEventListener[], onReconnect: () => Promise<void>): SLWebSocket {
         const instance = new SLWebSocket(url, buffer, listeners, onReconnect);
+        if (!instance) { console.error(`[Web Socket] Failed to initialize web socket.`); }
         SLWebSocket.instance = instance;
         return instance;
     }
@@ -92,7 +93,7 @@ export class SLWebSocket extends WebSocket {
                     this.sendBufferedMessages();
                 }).catch((error) => {
                     if (!this.isCurrent) { return; }
-                    console.log(`[Web Socket] Reconnection failed due to error: ${error}`);
+                    console.error(`[Web Socket] Reconnection failed due to error: ${error}`);
                     this.close(4000, `Reconnection failed due to error: ${error}`); // 4000-4999 are reserved for application use
                 });
             } else {
@@ -176,7 +177,7 @@ export class SLWebSocket extends WebSocket {
     }
 
     private onError(error: ErrorEvent) {
-        console.log(`[Web Socket] Error: ${error.message}.`);
+        console.error(`[Web Socket] Error: ${error.message}.`);
     }
 
     private onClose(e: CloseEvent) {