Parcourir la source

Handle the ‘connected’ kernel status differently.

Kernel info is requested by the onWSOpen function already, which is what sets the connected state, so no need to request the kernel info here too.
Jason Grout il y a 6 ans
Parent
commit
08474e98d4
1 fichiers modifiés avec 3 ajouts et 1 suppressions
  1. 3 1
      packages/services/src/kernel/default.ts

+ 3 - 1
packages/services/src/kernel/default.ts

@@ -818,7 +818,6 @@ export class DefaultKernel implements Kernel.IKernel {
           this._readyPromise.resolve();
         }
         break;
-      case 'connected':
       case 'restarting':
         // Send a kernel_info_request to get to a known kernel state.
         void this.requestKernelInfo().catch(() => {
@@ -832,6 +831,9 @@ export class DefaultKernel implements Kernel.IKernel {
         // the 'restarting' handler above is fine, so we do nothing here.
         /* no-op */
         break;
+      case 'connected':
+        // requestKernelInfo is sent by the onWSOpen
+        break;
       case 'reconnecting':
         if (this._isReady) {
           this._isReady = false;