Merge pull request #1028 from Microsoft/to-canonical-called-twice

Preventing toCanonical being called twice

Author: StanleyGoldman

src/authentication/keychain.ts

@@ -52,7 +52,7 @@ export function init(ctx: GlobalStateContext, keychain: Keytar = systemKeychain)
 
 export async function getToken(host: string, { storage = defaultStorage, keychain = defaultKeychain } = {}): Promise<string | null | undefined> {
 	host = toCanonical(host);
-	const token = keychain!.getPassword(SERVICE_ID, toCanonical(host))
+	const token = keychain!.getPassword(SERVICE_ID, host)
 		.catch(() => storage!.get(keyFor(host)));
 
 	// While we're transitioning everything out of configuration and into local storage, it's possible