Revoke token on logout (#541)

browser-extensions/chrome/src/App.tsx

@@ -11,7 +11,6 @@ import './styles/app.css';
 import EncryptionUtility from './utils/EncryptionUtility';
 import { VaultResponse } from './types/webapi/VaultResponse';
 import { useWebApi } from './context/WebApiContext';
-import SrpUtility from './utils/SrpUtility';
 
 /**
  * Main application component
@@ -76,11 +75,12 @@ const App: React.FC = () => {
    */
   const handleLogout = async (): Promise<void> => {
     setIsLoading(true);
+    setIsUserMenuOpen(false);
     try {
+      await webApi.logout();
       await authContext.logout();
     } finally {
       setIsLoading(false);
-      setIsUserMenuOpen(false);
     }
   };
 

browser-extensions/chrome/src/contentScript.ts

@@ -46,15 +46,20 @@ function createPopup(input: HTMLInputElement, credentials: Credential[]) : void
     padding: 8px 0;
   `;
 
-  // Change to mousedown event instead of click
-  const handleClickOutside = (event: MouseEvent) => {
+  /**
+   * Close autofill popup when clicking outside.
+   */
+  const handleClickOutside = (event: MouseEvent) : void => {
     if (!popup.contains(event.target as Node)) {
       removeExistingPopup();
       document.removeEventListener('mousedown', handleClickOutside);
     }
   };
 
-  // Use setTimeout to prevent immediate trigger of the mousedown event
+  /**
+   * Add event listener to document to close popup when clicking outside
+   * after a short delay to prevent immediate trigger of the mousedown event.
+   */
   setTimeout(() => {
     document.addEventListener('mousedown', handleClickOutside);
   }, 100);

browser-extensions/chrome/src/pages/Unlock.tsx

@@ -67,6 +67,7 @@ const Unlock: React.FC = () => {
   const handleLogout = async () : Promise<void> => {
     showLoading();
     try {
+      await webApi.logout();
       await authContext.logout();
     } finally {
       hideLoading();

browser-extensions/chrome/src/utils/WebApiService.ts

@@ -42,11 +42,12 @@ export class WebApiService {
   }
 
   /**
-   * Fetch data from the API
+   * Fetch data from the API.
    */
   public async fetch<T>(
     endpoint: string,
-    options: RequestInit = {}
+    options: RequestInit = {},
+    parseJson: boolean = true
   ): Promise<T> {
     const url = this.baseUrl + endpoint;
     const headers = new Headers(options.headers || {});
@@ -78,7 +79,7 @@ export class WebApiService {
             throw new Error('Request failed after token refresh');
           }
 
-          return retryResponse.json();
+          return parseJson ? retryResponse.json() : retryResponse.text() as T;
         } else {
           this.handleLogout();
           throw new Error('Session expired');
@@ -89,15 +90,15 @@ export class WebApiService {
         throw new Error(`HTTP error! status: ${response.status}`);
       }
 
-      return response.json();
+      return parseJson ? response.json() : response.text() as T;
     } catch (error) {
       console.error('API request failed:', error);
       throw error;
     }
   }
 
   /**
-   * Refresh the access token
+   * Refresh the access token.
    */
   private async refreshAccessToken(): Promise<string | null> {
     const refreshToken = this.getRefreshToken();
@@ -132,27 +133,31 @@ export class WebApiService {
   }
 
   /**
-   * Get a resource
+   * Issue GET request to the API.
    */
   public async get<T>(endpoint: string): Promise<T> {
     return this.fetch<T>(endpoint, { method: 'GET' });
   }
 
   /**
-   * Create a resource
+   * Issue POST request to the API.
    */
-  public async post<TRequest, TResponse>(endpoint: string, data: TRequest): Promise<TResponse> {
+  public async post<TRequest, TResponse>(
+    endpoint: string,
+    data: TRequest,
+    parseJson: boolean = true
+  ): Promise<TResponse> {
     return this.fetch<TResponse>(endpoint, {
       method: 'POST',
       headers: {
         'Content-Type': 'application/json',
       },
       body: JSON.stringify(data),
-    });
+    }, parseJson);
   }
 
   /**
-   * Update a resource
+   * Issue PUT request to the API.
    */
   public async put<TRequest, TResponse>(endpoint: string, data: TRequest): Promise<TResponse> {
     return this.fetch<TResponse>(endpoint, {
@@ -165,9 +170,24 @@ export class WebApiService {
   }
 
   /**
-   * Delete a resource
+   * Issue DELETE request to the API.
    */
   public async delete<T>(endpoint: string): Promise<T> {
     return this.fetch<T>(endpoint, { method: 'DELETE' });
   }
-}
+
+  /**
+   * Logout and revoke tokens via WebApi.
+   */
+  public async logout(): Promise<void> {
+    const refreshToken = this.getRefreshToken();
+    if (!refreshToken) {
+      return;
+    }
+
+    await this.post('Auth/revoke', {
+      token: this.getAccessToken(),
+      refreshToken: refreshToken,
+    }, false);
+  }
+}