Use registered auth providers instead of declared for getting label, f…

…ixes #117463
microsoft · Feb 23, 2021 · a5a8392 · a5a8392
1 parent 6fa58b5
commit a5a8392
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/vs/workbench/services/authentication/browser/authenticationService.ts b/src/vs/workbench/services/authentication/browser/authenticationService.ts
@@ -122,6 +122,7 @@ export interface IAuthenticationService {
 
 	readonly onDidChangeSessions: Event<{ providerId: string, label: string, event: AuthenticationSessionsChangeEvent }>;
 
+	// TODO @RMacfarlane completely remove this property
 	declaredProviders: AuthenticationProviderInformation[];
 	readonly onDidChangeDeclaredProviders: Event<AuthenticationProviderInformation[]>;
 
@@ -661,11 +662,11 @@ export class AuthenticationService extends Disposable implements IAuthentication
 		}
 	}
 	getLabel(id: string): string {
-		const authProvider = this.declaredProviders.find(provider => provider.id === id);
+		const authProvider = this._authenticationProviders.get(id);
 		if (authProvider) {
 			return authProvider.label;
 		} else {
-			throw new Error(`No authentication provider '${id}' has been declared.`);
+			throw new Error(`No authentication provider '${id}' is currently registered.`);
 		}
 	}