Fix handling of change of guacamole tunnel states: previously used th…

…e "open" tunnel state (1) to show the "connecting" screen - this should be the "connecting" tunnel state (0). After the unstable "state" is corrected, guacamole fires an "open" tunnel state which generated to presentation of the connecting screen. Fixes #3.
ILLGrenoble · Dec 9, 2024 · e8dc94a · e8dc94a
1 parent cd9084b
commit e8dc94a
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/src/app/user/instance/instance.component.ts b/src/app/user/instance/instance.component.ts
@@ -297,7 +297,6 @@ export class InstanceComponent implements OnInit, OnDestroy {
 
         } else {
             const tunnel = this.accountService.createGuacamoleRemoteDesktopTunnel(token, this.eventsGateway.clientId);
-            tunnel.unstableThreshold = 10000;
             this.manager = new GuacamoleVirtualDesktopManager(tunnel);
         }
         return true;

diff --git a/src/app/vdi/services/guacamole-virtual-desktop-manager.service.ts b/src/app/vdi/services/guacamole-virtual-desktop-manager.service.ts
@@ -223,7 +223,7 @@ export class GuacamoleVirtualDesktopManager extends VirtualDesktopManager {
     private handleTunnelStateChange(state: number): void {
         switch (state) {
             // Connection is being established
-            case 1:
+            case 0:
                 this.setState(VirtualDesktopManager.STATE.CONNECTING);
                 break;
             // Connection has closed