chore(Authorization)!: Updates Token Storage Structure

- Updates the structure of how tokens are stored on the configured storage system to allow retrieval by scope when necessary.
- **Allows for multiple tokens, for the same resource server, with different scopes to be stored.**

Author: jbottigliero

src/core/authorization/AuthorizationManager.ts

@@ -159,7 +159,6 @@ export class AuthorizationManager {
      * @event AuthorizationManager.events#authenticated
      * @type {object}
      * @property {boolean} isAuthenticated - Whether the `AuthorizationManager` is authenticated.
-     * @property {TokenResponse} [token] - The token response if the `AuthorizationManager` is authenticated.
      */
     authenticated: new Event<
       'authenticated',
@@ -169,7 +168,6 @@ export class AuthorizationManager {
          * @see {@link AuthorizationManager.authenticated}
          */
         isAuthenticated: boolean;
-        token?: TokenResponse;
       }
     >('authenticated'),
     /**
@@ -239,8 +237,12 @@ export class AuthorizationManager {
    * @see {@link https://docs.globus.org/api/auth/reference/#oidc_userinfo_endpoint}
    */
   get user() {
-    const token = this.getGlobusAuthToken();
-    return token && token.id_token ? jwtDecode<JwtUserInfo>(token.id_token) : null;
+    const token = this.tokens
+      .getAll()
+      .find((t) => t.resource_server === RESOURCE_SERVERS.AUTH && t.scope.includes('openid'));
+    return token && 'id_token' in token && token.id_token
+      ? jwtDecode<JwtUserInfo>(token.id_token)
+      : null;
   }
 
   /**
@@ -289,32 +291,28 @@ export class AuthorizationManager {
 
   /**
    * Whether or not the instance has a reference to a Globus Auth token.
+   * @deprecated Use `AuthorizationManager.tokens.auth` instead.
    */
   hasGlobusAuthToken() {
-    return this.getGlobusAuthToken() !== null;
+    return Boolean(this.tokens.auth);
   }
 
   /**
    * Retrieve the Globus Auth token managed by the instance.
+   * @deprecated Use `AuthorizationManager.tokens.auth` instead.
    */
   getGlobusAuthToken() {
-    const entry = this.storage.getItem(`${this.storageKeyPrefix}${RESOURCE_SERVERS.AUTH}`);
-    return entry ? JSON.parse(entry) : null;
+    return this.tokens.auth;
   }
 
   #checkAuthorizationState() {
     log('debug', 'AuthorizationManager.#checkAuthorizationState');
-    if (this.hasGlobusAuthToken()) {
-      this.authenticated = true;
-    }
+    this.authenticated = Boolean(this.tokens.auth);
   }
 
   async #emitAuthenticatedState() {
-    const isAuthenticated = this.authenticated;
-    const token = this.getGlobusAuthToken() ?? undefined;
     await this.events.authenticated.dispatch({
-      isAuthenticated,
-      token,
+      isAuthenticated: this.authenticated,
     });
   }
 

src/core/authorization/TokenManager.ts

@@ -1,11 +1,24 @@
+/* eslint-disable no-underscore-dangle */
 import { CONFIG, isToken } from '../../services/auth/index.js';
 
 import { SERVICES, type Service } from '../global.js';
+import { log } from '../logger.js';
 import { AuthorizationManager } from './AuthorizationManager.js';
 
 import type { Token, TokenResponse } from '../../services/auth/types.js';
 
-export type StoredToken = Token & {
+const TOKEN_STORAGE_VERSION = 0;
+
+type TokenStorage = {
+  version: typeof TOKEN_STORAGE_VERSION;
+  state: {
+    tokens: Record<StoredToken['access_token'], StoredToken>;
+  };
+};
+
+type ByScopeCache = Record<string, StoredToken['access_token']>;
+
+export type StoredToken = (Token | TokenResponse) & {
   /**
    * Tokens stored before the introduction of the `__metadata` field will be missing this property.
    * @since 4.3.0
@@ -28,34 +41,127 @@ export type StoredToken = Token & {
 export class TokenManager {
   #manager: AuthorizationManager;
 
+  #storageKey: string;
+
+  #byScopeCache: ByScopeCache = {};
+
   constructor(options: { manager: AuthorizationManager }) {
     this.#manager = options.manager;
+    this.#storageKey = `${this.#manager.storageKeyPrefix}TokenManager`;
+    this.#migrate();
+  }
+
+  get #hasStorage() {
+    return this.#manager.storage.getItem(this.#storageKey) !== null;
+  }
+
+  get #storage(): TokenStorage {
+    const raw = this.#manager.storage.getItem(this.#storageKey);
+    if (!raw) {
+      throw new Error('@globus/sdk | Unable to retrieve TokenStorage.');
+    }
+    return JSON.parse(raw);
+  }
+
+  set #storage(value: TokenStorage) {
+    this.#manager.storage.setItem(this.#storageKey, JSON.stringify(value));
+    /**
+     * When the storage is update, we need to rebuild the cache of tokens by scope.
+     */
+    this.#byScopeCache = Object.values(value.state.tokens).reduce((acc: ByScopeCache, token) => {
+      token.scope.split(' ').forEach((scope) => {
+        /**
+         * If there isn't an existing token for the scope, add it to the cache.
+         */
+        if (!acc[scope]) {
+          acc[scope] = token.access_token;
+          return;
+        }
+        /**
+         * If there is an existing token for the scope, compare the expiration times and keep the token that expires later.
+         */
+        const existing = value.state.tokens[acc[scope]];
+        /**
+         * If the existing token or the new token is missing the expiration metadata, skip the comparison.
+         */
+        if (!existing.__metadata?.expires || !token.__metadata?.expires) {
+          return;
+        }
+        if (existing.__metadata.expires < token.__metadata.expires) {
+          acc[scope] = token.access_token;
+        }
+      });
+      return acc;
+    }, {});
   }
 
   /**
-   * Retrieve and parse an item from the storage.
+   * Migrates the token storage to the latest version (if necessary).
    */
-  #getTokenFromStorage(key: string) {
-    const raw = this.#manager.storage.getItem(key) || 'null';
-    let token: StoredToken | null = null;
-    try {
-      const parsed = JSON.parse(raw);
-      if (isToken(parsed)) {
-        token = parsed;
-      }
-    } catch (e) {
-      // no-op
+  #migrate() {
+    if (this.#hasStorage && this.#storage.version === TOKEN_STORAGE_VERSION) {
+      /**
+       * Storage entry exists and matches the current version.
+       */
+      return;
     }
-    return token;
+    /**
+     * Migrate legacy token storage to the new format.
+     *
+     * Tokens were previously stored as individual items in the storage with keys that
+     * included the resource server, e.g. `{client_id}:auth.globus.org`
+     */
+    const tokens: TokenStorage['state']['tokens'] = {};
+    Object.keys(this.#manager.storage).forEach((key) => {
+      if (key.startsWith(this.#manager.storageKeyPrefix)) {
+        const maybeToken = this.#manager.storage.getItem(key);
+        if (isToken(maybeToken)) {
+          tokens[maybeToken.access_token] = maybeToken;
+        }
+      }
+    }, {});
+    this.#storage = {
+      version: TOKEN_STORAGE_VERSION,
+      state: {
+        tokens,
+      },
+    };
   }
 
-  #getTokenForService(service: Service) {
+  #getTokenForService(service: Service): StoredToken | null {
     const resourceServer = CONFIG.RESOURCE_SERVERS?.[service];
     return this.getByResourceServer(resourceServer);
   }
 
-  getByResourceServer(resourceServer: string): StoredToken | null {
-    return this.#getTokenFromStorage(`${this.#manager.storageKeyPrefix}${resourceServer}`);
+  /**
+   * Retrieve a token by the `resource_server` and optional `scope`. If a `scope` is provided, the token will be retrieved by the scope.
+   * This is useful when your application needs to manage multiple tokens for the same `resource_server`, but with different scopes.
+   *
+   * **IMPORTANT**: If multiple tokens are found for the same `resource_server` (and no `scope` is provided), the first identified token will be returned.
+   * If your application requires multiple tokens for the same `resource_server` this might lead to unexpected behavior (e.g. using the wrong token for requests).
+   * In this case, you can use the `scope` parameter to retrieve the token you need, or use the `getAllByResourceServer` method to retrieve all tokens for a `resource_server`
+   * and manage them as needed.
+   */
+  getByResourceServer(resourceServer: string, scope?: string) {
+    if (scope) {
+      return this.getByScope(scope);
+    }
+    const tokens = this.getAllByResourceServer(resourceServer);
+    if (tokens.length > 1) {
+      log(
+        'warn',
+        `TokenManager.getByResource | Multiple tokens found for resource server, narrow your token selection by providing a "scope" parameter. | resource_server=${resourceServer}`,
+      );
+    }
+    return tokens.length ? tokens[0] : null;
+  }
+
+  getAllByResourceServer(resourceServer: string): StoredToken[] {
+    return this.getAll().filter((token) => token.resource_server === resourceServer);
+  }
+
+  getByScope(scope: string): StoredToken | null {
+    return this.#storage.state.tokens[this.#byScopeCache[scope]] || null;
   }
 
   get auth(): StoredToken | null {
@@ -90,45 +196,84 @@ export class TokenManager {
     return this.getByResourceServer(endpoint);
   }
 
+  /**
+   * Retrieve all tokens from the storage.
+   */
   getAll(): StoredToken[] {
-    const entries = Object.keys(this.#manager.storage).reduce(
-      (acc: (StoredToken | null)[], key) => {
-        if (key.startsWith(this.#manager.storageKeyPrefix)) {
-          acc.push(this.#getTokenFromStorage(key));
-        }
-        return acc;
-      },
-      [],
-    );
-    return entries.filter(isToken);
+    return Object.values(this.#storage?.state.tokens);
   }
 
   /**
    * Add a token to the storage.
    */
   add(token: Token | TokenResponse) {
+    if (!isToken(token)) {
+      throw new Error('@globus/sdk | Invalid token provided to TokenManager.add');
+    }
     const created = Date.now();
     const expires = created + token.expires_in * 1000;
-    this.#manager.storage.setItem(
-      `${this.#manager.storageKeyPrefix}${token.resource_server}`,
-      JSON.stringify({
-        ...token,
-        /**
-         * Add metadata to the token to track when it was created and when it expires.
-         */
-        __metadata: {
-          created,
-          expires,
+    const storage = this.#storage;
+    /**
+     * When adding a token, we **replace** any existing tokens with the same `resource_server` and `scope`
+     * by filtering them out of the storage before adding the new token.
+     */
+    const tokens = Object.entries(storage.state.tokens).reduce((acc, [key, value]) => {
+      if (value.resource_server === token.resource_server && value.scope === token.scope) {
+        return acc;
+      }
+      return {
+        ...acc,
+        [key]: value,
+      };
+    }, {});
+    this.#storage = {
+      ...storage,
+      state: {
+        tokens: {
+          ...tokens,
+          [token.access_token]: {
+            ...token,
+            /**
+             * Add metadata to the token to track when it was created and when it expires.
+             */
+            __metadata: {
+              created,
+              expires,
+            },
+          },
         },
-      }),
-    );
+      },
+    };
     if ('other_tokens' in token) {
       token.other_tokens?.forEach((t) => {
         this.add(t);
       });
     }
   }
 
+  remove(token: Token) {
+    const storage = this.#storage;
+    if (!storage) {
+      return;
+    }
+    delete storage.state.tokens[token.access_token];
+    this.#storage = {
+      ...storage,
+      state: {
+        tokens: storage.state.tokens,
+      },
+    };
+  }
+
+  clear() {
+    this.#storage = {
+      version: TOKEN_STORAGE_VERSION,
+      state: {
+        tokens: {},
+      },
+    };
+  }
+
   /**
    * Determines whether or not a stored token is expired.
    * @param token The token to check.
@@ -137,11 +282,9 @@ export class TokenManager {
    * based on the token's metadata. This can happen if the token is missing the `__metadata` field or the `expires` field.
    */
   static isTokenExpired(token: StoredToken | null, augment: number = 0): boolean | undefined {
-    /* eslint-disable no-underscore-dangle */
     if (!token || !token.__metadata || typeof token.__metadata.expires !== 'number') {
       return undefined;
     }
     return Date.now() + augment >= token.__metadata.expires;
-    /* eslint-enable no-underscore-dangle */
   }
 }