[Bug] MSAL.NET throws a MsalClientException and not a MsalServiceException for the reset password exception from AAD B2C

Which Version of MSAL are you using ? First noticed in 4.1, but not sure which version introduced the change in behavior

Platform mobile, desktop, web app

Repro

example, include this code:

catch (MsalServiceException ex)
          {
                try
                {
                          if (ex.Message.Contains("AADB2C90118"))
                          {
                                   authResult = await (app as 
                                         publicClientApplication).AcquireTokenInteractive(App.ApiScopes)
                                        .WithParentActivityOrWindow(new WindowInteropHelper(this).Handle)
                                        .WithAccount(GetAccountByPolicy(accounts, App.PolicySignUpSignIn))
                                        .WithPrompt(Prompt.SelectAccount)
                                        .WithB2CAuthority(App.AuthorityResetPassword)
                                        .ExecuteAsync();
}

Can be tried w/B2C desktop sample.

Expected behavior MSAL.NET should throw a MsalServiceException with the following error message: AADB2C90118: The user has forgotten their password.

Actual behavior MSAL.NET throws a MsalClientException instead

[jmprieur]

yes agree. even if this is a behavioral breaking change.

[henrik-me]

@jennyf19 @jmprieur : I suppose we should consider fixing this as it’s a regression betwen 3 and 4. 4.3 seems like a good vehicle for this change imo.