Merge pull request #875 from AzureAD/avdunn/expire-buffer-fix

Correctly set buffer for expired tokens
AzureAD · Oct 27, 2024 · c7ed174 · c7ed174
2 parents ff44d4a + 412c842
commit c7ed174
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/msal4j-sdk/src/main/java/com/microsoft/aad/msal4j/AcquireTokenSilentSupplier.java b/msal4j-sdk/src/main/java/com/microsoft/aad/msal4j/AcquireTokenSilentSupplier.java
@@ -129,7 +129,7 @@ private boolean shouldRefresh(SilentParameters parameters, AuthenticationResult
         long currTimeStampSec = new Date().getTime() / 1000;
 
         //If the access token is expired or within 5 minutes of becoming expired, refresh it
-        if (!StringHelper.isBlank(cachedResult.accessToken()) && cachedResult.expiresOn() < (currTimeStampSec - ACCESS_TOKEN_EXPIRE_BUFFER_IN_SEC)) {
+        if (!StringHelper.isBlank(cachedResult.accessToken()) && cachedResult.expiresOn() < (currTimeStampSec + ACCESS_TOKEN_EXPIRE_BUFFER_IN_SEC)) {
             setCacheTelemetry(CacheTelemetry.REFRESH_ACCESS_TOKEN_EXPIRED.telemetryValue);
             log.debug("Refreshing access token because it is expired.");
             return true;