Add closed bool to prevent reconnection (#361)

Author: njooma

examples/vanilla/package-lock.json

@@ -52,6 +52,7 @@ const connect = async () => {
       credential: {
         type: 'api-key',
         payload: API_KEY,
+        authEntity: API_KEY_ID,
       },
       authEntity: API_KEY_ID,
       signalingAddress: 'https://app.viam.com:443',

examples/vanilla/src/main.ts

@@ -99,6 +99,8 @@ export class RobotClient extends EventDispatcher implements Robot {
 
   private savedCreds: Credentials | undefined;
 
+  private closed: boolean;
+
   private robotServiceClient: RobotServiceClient | undefined;
 
   private armServiceClient: ArmServiceClient | undefined;
@@ -182,6 +184,8 @@ export class RobotClient extends EventDispatcher implements Robot {
         this.emit('connectionstatechange', { eventType });
       });
     }
+
+    this.closed = false;
   }
 
   private onDisconnect(event?: Event) {
@@ -191,6 +195,10 @@ export class RobotClient extends EventDispatcher implements Robot {
       return;
     }
 
+    if (this.closed) {
+      return;
+    }
+
     // eslint-disable-next-line no-console
     console.debug('Connection closed, will try to reconnect');
     void backOff(async () => this.connect(), {
@@ -395,6 +403,7 @@ export class RobotClient extends EventDispatcher implements Robot {
       this.peerConn = undefined;
     }
     this.sessionManager.reset();
+    this.closed = true;
     this.emit(MachineConnectionEvent.DISCONNECTED, {});
   }
 
@@ -411,6 +420,7 @@ export class RobotClient extends EventDispatcher implements Robot {
     dialTimeout,
   }: ConnectOptions = {}) {
     this.emit(MachineConnectionEvent.CONNECTING, {});
+    this.closed = false;
 
     if (this.connecting) {
       // This lint is clearly wrong due to how the event loop works such that after an await, the condition may no longer be true.