Use Promise.allSettled in deactivate, and add this logic to web-extension

Author: gjsjohnmurray

src/extension.ts

@@ -23,7 +23,10 @@ export function activate(context: vscode.ExtensionContext) {
 
 export async function deactivate() {
 	// Do our best to log out of all sessions
+
+	const promises: Promise<any>[] = [];
 	for (const serverSession of serverSessions) {
-		await logout(serverSession[1].serverName);
+		promises.push(logout(serverSession[1].serverName));
 	}
+	await Promise.allSettled(promises);
 }

src/web-extension.ts

@@ -3,12 +3,21 @@
 import * as vscode from "vscode";
 import { ServerManagerView } from "./ui/serverManagerView";
 import { commonActivate } from "./commonActivate";
+import { logout, serverSessions } from "./makeRESTRequest";
 
 export function activate(context: vscode.ExtensionContext) {
-    const view = new ServerManagerView(context);
+	const view = new ServerManagerView(context);
 
-    // Common activation steps
-    return commonActivate(context, view);
+	// Common activation steps
+	return commonActivate(context, view);
 }
 
-export function deactivate() { }
+export async function deactivate() {
+	// Do our best to log out of all sessions
+
+	const promises: Promise<any>[] = [];
+	for (const serverSession of serverSessions) {
+		promises.push(logout(serverSession[1].serverName));
+	}
+	await Promise.allSettled(promises);
+}