Fix for empty scopes. HasScopes does return true when empty scopes ar…

…e passed (#3677) * fix for empty scopes. HasScopes does return true when empty scopes are passed * null check Co-authored-by: Gladwin Johnson <[email protected]>
AzureAD · Sep 16, 2022 · 8d9786b · 8d9786b
1 parent 75de81e
commit 8d9786b
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/client/Microsoft.Identity.Client.Broker/WamAdapters.cs b/src/client/Microsoft.Identity.Client.Broker/WamAdapters.cs
@@ -322,7 +322,9 @@ public static void ValidateAuthParams(
         {
             //MSAL Runtime throws an ApiContractViolation Exception with Tag: 0x2039c1cb (InvalidArg)
             //When no scopes are passed, this will check if user is passing scopes
-            if (!authenticationRequestParameters.HasScopes)
+            var scopes = authenticationRequestParameters.Scope?.ElementAtOrDefault(0);
+
+            if (string.IsNullOrWhiteSpace(scopes))
             {
                 logger.Error($"[WamBroker] {MsalError.WamScopesRequired} " +
                     $"{MsalErrorMessage.ScopesRequired}");