Fixed bug where user-supplied cache is loaded into memory before look…

…ing into the cache
AzureAD · Jul 6, 2023 · 9750178 · 9750178
1 parent fb446dc
commit 9750178
Showing 1 changed file with 11 additions and 0 deletions.
diff --git a/lib/msal-node/src/client/ClientCredentialClient.ts b/lib/msal-node/src/client/ClientCredentialClient.ts
@@ -25,6 +25,7 @@ import {
     ServerAuthorizationTokenResponse,
     StringUtils,
     TimeUtils,
+    TokenCacheContext,
     UrlString,
 } from "@azure/msal-common";
 
@@ -71,6 +72,16 @@ export class ClientCredentialClient extends BaseClient {
     private async getCachedAuthenticationResult(
         request: CommonClientCredentialRequest
     ): Promise<AuthenticationResult | null> {
+        // read the user-supplied cache into memory, if applicable
+        if (this.config.serializableCache && this.config.persistencePlugin) {
+            const cacheContext = new TokenCacheContext(
+                this.config.serializableCache,
+                true,
+            );
+
+            await this.config.persistencePlugin.beforeCacheAccess(cacheContext);
+        }
+
         const cachedAccessToken = this.readAccessTokenFromCache();
 
         if (!cachedAccessToken) {