improved connection handling (#656)

- improved cleanup on disconnect
  - fixed user follow failing after token refresh
  - set default RetryConnectionInterval to 0 to avoid issues after disconnects

src/app/components/wmks/wmks.component.ts

@@ -67,19 +67,16 @@ export class WmksComponent implements OnInit, OnDestroy {
     if (this.connectTimerSubscription != null) {
       this.connectTimerSubscription.unsubscribe();
     }
+
+    this.vmService.disconnect();
   }
 
   private setVmId(value: string) {
     if (this.connectTimerSubscription != null) {
       this.connectTimerSubscription.unsubscribe();
     }
 
-    // destroy and re-create the wmksContainer
-    this.showWmks = false;
-    this.showWmks = true;
-
     this._vmId = value;
-
     this.vmService.disconnect();
   }
 

src/app/services/signalr/signalr.service.ts

@@ -22,6 +22,7 @@ export class SignalRService {
 
   private userId: string;
   private viewId: string;
+  private activeVmId: string;
 
   private apiUrl: string;
 
@@ -78,10 +79,15 @@ export class SignalRService {
     if (this.userId && this.viewId) {
       this.joinUser(this.userId, this.viewId);
     }
+
+    if (this.activeVmId) {
+      this.setActiveVirtualMachine(this.activeVmId);
+    }
   }
 
   public joinUser(userId: string, viewId: string) {
     this.userId = userId;
+    this.viewId = viewId;
 
     this.startConnection().then(() => {
       this.hubConnection.invoke('JoinUser', userId, viewId).then((vmUser) => {
@@ -100,12 +106,14 @@ export class SignalRService {
   }
 
   public setActiveVirtualMachine(vmId: string) {
+    this.activeVmId = vmId;
     this.startConnection().then(() => {
       this.hubConnection.invoke('SetActiveVirtualMachine', vmId);
     });
   }
 
   public unsetActiveVirtualMachine() {
+    this.activeVmId = null;
     this.startConnection().then(() => {
       this.hubConnection.invoke('UnsetActiveVirtualMachine');
     });

src/app/state/vsphere/vsphere.service.ts

@@ -387,9 +387,9 @@ export class VsphereService {
 
   public disconnect() {
     if (this.wmks != null) {
-      this.wmks.destroy();
       this.wmks.unregister();
-      // this.wmks.disconnect();
+      this.wmks.disconnect();
+      this.wmks.destroy();
       this.wmks = null;
     }
 

src/assets/config/settings.json

@@ -36,6 +36,6 @@
     { "name": "Slowest", "value": 500 }
   ],
   "WMKS": {
-    "RetryConnectionInterval": 5000
+    "RetryConnectionInterval": 0
   }
 }