Persist sessions per gitpod host

Author: jeanp413

extensions/gitpod/src/authentication.ts

@@ -36,20 +36,20 @@ export default class GitpodAuthenticationProvider extends Disposable implements
 
 		this._logger = logger;
 		this._telemetry = telemetry;
-		this._keychain = new Keychain(this.context, `gitpod.auth`, this._logger);
 
-		const serviceUrl = vscode.workspace.getConfiguration('gitpod').get<string>('host')!;
-		this._gitpodServer = new GitpodServer(serviceUrl, this._logger);
-		this._register(vscode.workspace.onDidChangeConfiguration(async e => {
+		const gitpodHost = vscode.workspace.getConfiguration('gitpod').get<string>('host')!;
+		const serviceUrl = new URL(gitpodHost);
+		this._gitpodServer = new GitpodServer(serviceUrl.toString(), this._logger);
+		this._keychain = new Keychain(this.context, `gitpod.auth.${serviceUrl.hostname}`, this._logger);
+		this._register(vscode.workspace.onDidChangeConfiguration(e => {
 			if (e.affectsConfiguration('gitpod.host')) {
-				const serviceUrl = vscode.workspace.getConfiguration('gitpod').get<string>('host')!;
+				const gitpodHost = vscode.workspace.getConfiguration('gitpod').get<string>('host')!;
+				const serviceUrl = new URL(gitpodHost);
 				this._gitpodServer.dispose();
-				this._gitpodServer = new GitpodServer(serviceUrl, this._logger);
+				this._gitpodServer = new GitpodServer(serviceUrl.toString(), this._logger);
+				this._keychain = new Keychain(this.context, `gitpod.auth.${serviceUrl.hostname}`, this._logger);
 
-				// Remove all sessions when gitpod.host changes
-				const sessions = await this._sessionsPromise;
-				await this._keychain.deleteToken();
-				this._sessionChangeEmitter.fire({ added: [], removed: sessions, changed: [] });
+				this.checkForUpdates();
 			}
 		}));
 

extensions/gitpod/src/extension.ts

@@ -25,9 +25,16 @@ export async function activate(context: vscode.ExtensionContext) {
 
 	/* Gitpod settings sync */
 	await updateSyncContext();
-	context.subscriptions.push(vscode.workspace.onDidChangeConfiguration(e => {
+	context.subscriptions.push(vscode.workspace.onDidChangeConfiguration(async e => {
 		if (e.affectsConfiguration('gitpod.host') || e.affectsConfiguration('configurationSync.store')) {
-			updateSyncContext();
+			const addedSyncProvider = await updateSyncContext();
+			if (!addedSyncProvider) {
+				const action = 'Settings Sync: Enable signing in with Gitpod';
+				const result = await vscode.window.showInformationMessage('Gitpod Settings Sync configuration invalidated, Settings Sync is disabled.', action);
+				if (result === action) {
+					vscode.commands.executeCommand('gitpod.syncProvider.add');
+				}
+			}
 		}
 	}));
 

extensions/gitpod/src/gitpodServer.ts

@@ -40,7 +40,7 @@ export class GitpodServer extends Disposable {
 	constructor(serviceUrl: string, private readonly _logger: Log) {
 		super();
 
-		this._serviceUrl = new URL(serviceUrl).toString().replace(/\/$/, '');
+		this._serviceUrl = serviceUrl.replace(/\/$/, '');
 	}
 
 	public async login(scopes: string): Promise<string> {

extensions/gitpod/src/settingsSync.ts

@@ -36,13 +36,14 @@ function getGitpodSyncProviderConfig(serviceUrl: string): ConfigurationSyncStore
 /**
  * Updates the VS Code context to reflect whether the user added Gitpod as their Settings Sync provider.
  */
-export async function updateSyncContext() {
+export async function updateSyncContext(): Promise<boolean> {
 	const config = vscode.workspace.getConfiguration();
 	const syncProviderConfig = config.get('configurationSync.store');
 	const serviceUrl = config.get<string>('gitpod.host')!;
 	const gitpodSyncProviderConfig = getGitpodSyncProviderConfig(serviceUrl);
-	const addedSyncProvider = syncProviderConfig && JSON.stringify(syncProviderConfig) === JSON.stringify(gitpodSyncProviderConfig);
-	await vscode.commands.executeCommand('setContext', 'gitpod.addedSyncProvider', !!addedSyncProvider);
+	const addedSyncProvider = !!syncProviderConfig && JSON.stringify(syncProviderConfig) === JSON.stringify(gitpodSyncProviderConfig);
+	await vscode.commands.executeCommand('setContext', 'gitpod.addedSyncProvider', addedSyncProvider);
+	return addedSyncProvider;
 }
 
 /**