search

AzureAD / microsoft-authentication-library-for-java

Microsoft Authentication Library (MSAL) for Java http://aka.ms/aadv2
MIT License
285 stars 142 forks source link

Reduce logging level of CACHE_MISS <GitHub Issue #833> #844

Closed papaya2k closed 2 months ago

papaya2k commented 2 months ago

Addresses GitHub Issue #833 by reducing the logging level of a CACHE_MISS in the AuthenticationResultSupplier to debug level.

Avery-Dunn:

I agree that a cache miss exception should cause a debug log, since it doesn't necessarily mean that something wrong. It'll be a quick fix so will likely be in our next release, and I'll update this thread once we have some ETAs

Without this patch, the logging output is overly verbose about CACHE_MISS events. This should resolve that issue by changing the logging level.

Avery-Dunn commented 2 months ago

Looks good to me @papaya2k, thanks for fixing this!