Merge pull request #22 from GitGuardian/salomevoltz/add-logout-command

feat(auth-logout): Add logout command

Author: fnareoh

package.json

@@ -50,6 +50,10 @@
         "command": "gitguardian.authenticate",
         "title": "gitguardian: Authenticate"
       },
+      {
+        "command": "gitguardian.logout",
+        "title": "gitguardian: logout"
+      },
       {
         "command": "gitguardian.showOutput",
         "title": "Show Output"

src/extension.ts

@@ -5,6 +5,7 @@ import {
   ignoreLastFound,
   ignoreSecret,
   loginGGShield,
+  logoutGGShield,
   showAPIQuota,
 } from "./lib/ggshield-api";
 import {
@@ -255,6 +256,16 @@ export function activate(context: ExtensionContext) {
           } else {
             updateStatusBarItem(StatusBarStatus.unauthenticated, statusBar);
           }
+        }),
+        commands.registerCommand("gitguardian.logout", async () => {
+          outputChannel.show();
+          logoutGGShield(ggshieldResolver.configuration);
+          updateStatusBarItem(StatusBarStatus.unauthenticated, statusBar);
+          commands.executeCommand('setContext', 'isAuthenticated', false);
+          setApiKey(configuration, undefined);
+          ggshieldViewProvider.refresh();
+          ggshieldRemediationMessageViewProvider.refresh();
+          ggshieldQuotaViewProvider.refresh();
         })
       );
     })

src/lib/ggshield-api.ts

@@ -251,6 +251,12 @@ export async function loginGGShield(
   });
 }
 
+export function logoutGGShield(
+  configuration: GGShieldConfiguration
+): void {
+  const proc = runGGShieldCommand(configuration, ["auth", "logout"]);
+}
+
 export function ggshieldAuthStatus(
   configuration: GGShieldConfiguration
 ): boolean {

src/lib/ggshield-configuration.ts

@@ -40,10 +40,6 @@ export function getConfiguration(
 
 export function setApiKey(configuration: GGShieldConfiguration, apiKey: string | undefined): void {
   const config = workspace.getConfiguration("gitguardian");
-  if (!apiKey) {
-    throw new Error("Missing API Key");
-  }
-
-  configuration.apiKey = apiKey;
+  configuration.apiKey = apiKey ? apiKey : "";
   config.update("apiKey", apiKey, ConfigurationTarget.Global);
 }