Merge pull request #844 from papaya2k/cache-miss-logging-fix

Reduce logging level of CACHE_MISS <GitHub Issue #833>
AzureAD · Jul 25, 2024 · 006abce · 006abce
2 parents 9374a3a + 361ec5f
commit 006abce
Showing 1 changed file with 11 additions and 4 deletions.
diff --git a/msal4j-sdk/src/main/java/com/microsoft/aad/msal4j/AuthenticationResultSupplier.java b/msal4j-sdk/src/main/java/com/microsoft/aad/msal4j/AuthenticationResultSupplier.java
@@ -95,10 +95,17 @@ public IAuthenticationResult get() {
                         msalRequest.requestContext().correlationId(),
                         error);
 
-                clientApplication.log.warn(
-                        LogHelper.createMessage(
-                                String.format("Execution of %s failed: %s", this.getClass(), ex.getMessage()),
-                                msalRequest.headers().getHeaderCorrelationIdValue()));
+                String logMessage = LogHelper.createMessage(
+                        String.format("Execution of %s failed: %s", this.getClass(), ex.getMessage()),
+                        msalRequest.headers().getHeaderCorrelationIdValue());
+                if (ex instanceof MsalClientException) {
+                    MsalClientException exception = (MsalClientException) ex;
+                    if (exception.errorCode() != null && exception.errorCode().equalsIgnoreCase(AuthenticationErrorCode.CACHE_MISS)) {
+                        clientApplication.log.debug(logMessage);
+                    }
+                } else {
+                    clientApplication.log.warn(logMessage);
+                }
 
                 throw new CompletionException(ex);
             }