auth: Add check in getTenants (#1822)

microsoft · Nov 19, 2024 · e43f88f · e43f88f
1 parent 0ecfad7
commit e43f88f
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/auth/src/VSCodeAzureSubscriptionProvider.ts b/auth/src/VSCodeAzureSubscriptionProvider.ts
@@ -64,10 +64,14 @@ export class VSCodeAzureSubscriptionProvider extends vscode.Disposable implement
         const results: TenantIdDescription[] = [];
 
         for await (account of account ? [account] : await vscode.authentication.getAccounts(getConfiguredAuthProviderId())) {
-            const { client } = await this.getSubscriptionClient(account, undefined, undefined);
+            // Added check. Without this the getSubscriptionClient function throws the NotSignedInError
+            if (await this.isSignedIn(undefined, account)) {
 
-            for await (const tenant of client.tenants.list()) {
-                results.push(tenant);
+                const { client } = await this.getSubscriptionClient(account, undefined, undefined);
+
+                for await (const tenant of client.tenants.list()) {
+                    results.push(tenant);
+                }
             }
         }