Pick up better fix for event looping (#1279)

Author: bwateratmsft

package-lock.json

@@ -944,7 +944,7 @@
     "dependencies": {
         "@azure/arm-resources": "^5.2.0",
         "@azure/arm-resources-profile-2020-09-01-hybrid": "^2.1.0",
-        "@microsoft/vscode-azext-azureauth": "^5.1.0",
+        "@microsoft/vscode-azext-azureauth": "^5.1.1",
         "@microsoft/vscode-azext-azureutils": "^4.0.0",
         "@microsoft/vscode-azext-utils": "^4.0.0",
         "buffer": "^6.0.3",

package.json

@@ -55,20 +55,20 @@ export abstract class ResourceTreeDataProviderBase extends vscode.Disposable imp
         } else {
             if (!this.subscriptionProvider) {
                 this.subscriptionProvider = await ext.subscriptionProviderFactory();
-            }
 
-            if (!inCloudShell()) {
-                // Only outside of Cloud Shell will we monitor for ongoing session changes
-                // Inside we must avoid due to https://github.com/microsoft/vscode-dev/issues/1334
-                const one = this.subscriptionProvider.onDidSignIn(() => {
-                    this.notifyTreeDataChanged();
-                });
+                if (!inCloudShell()) {
+                    // Only outside of Cloud Shell will we monitor for ongoing session changes
+                    // Inside we must avoid due to https://github.com/microsoft/vscode-dev/issues/1334
+                    const one = this.subscriptionProvider.onDidSignIn(() => {
+                        this.notifyTreeDataChanged();
+                    });
 
-                const two = this.subscriptionProvider.onDidSignOut(() => {
-                    this.notifyTreeDataChanged();
-                });
+                    const two = this.subscriptionProvider.onDidSignOut(() => {
+                        this.notifyTreeDataChanged();
+                    });
 
-                this.statusSubscription = vscode.Disposable.from(one, two);
+                    this.statusSubscription = vscode.Disposable.from(one, two);
+                }
             }
 
             return this.subscriptionProvider;