[M] CCA cache - Log error when cache is not serialized

[pmaytak]

For confidential client apps, log error when cache is not serialized.

[jmprieur]

@pmaytak, ,

• we should log an error (but not throw)
• the error message should be: The default token cache provided by MSAL is not designed to be performant when used in confidential client applications. Please use token cache serialization. See https://aka.ms/msal-net-cca-token-cache-serialization.

[BobBonser]

I am getting this error in my log even though I do have AddInMemoryCaches in my startup

        var initialScopes = Configuration.GetValue<string>("DownstreamApi:Scopes")?.Split(' ');

        services.AddMicrosoftIdentityWebAppAuthentication(Configuration, "AzureAd")
            .EnableTokenAcquisitionToCallDownstreamApi(initialScopes)
                .AddMicrosoftGraph(Configuration.GetSection("DownstreamApi"))
                .AddInMemoryTokenCaches();

[pmaytak]

Thanks @BobBonser, Is this the same issue as #2990?

[jmprieur]

Which version of Id.Web do you use. @BobBonser ?

[BobBonser]

It does appear to be the same issue.

…

Sent from my Pixel 4

On Fri, Nov 5, 2021, 3:34 PM pmaytak ***@***.***> wrote: Thanks @BobBonser <https://github.com/BobBonser>, Is this the same issue as #2990 <#2990> ? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#2461 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AALGYZBZOVYER22733E3HOLUKQ5VRANCNFSM4Y23AGLQ> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[BobBonser]

Not sure what I'd.Web means.

…

Sent from my Pixel 4

On Fri, Nov 5, 2021, 9:32 PM Jean-Marc Prieur ***@***.***> wrote: Which version of Id.Web do you use. @BobBonser <https://github.com/BobBonser> ? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#2461 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AALGYZE5UYJEXVMP5SGBB3LUKSHT3ANCNFSM4Y23AGLQ> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[bgavrilMS]

@BobBonser - Microsoft.Identity.Web ... But let's continue the conversation of #2990, as this issue is closed.