Merge pull request #2073 from microsoftgraph/bugfixes/UseAuthority

Use authorityUrl instead of tenantId when initializing MSAL

Author: peombwa

src/Authentication/Authentication.Core/Utilities/AuthenticationHelpers.cs

@@ -23,7 +23,7 @@ namespace Microsoft.Graph.PowerShell.Authentication.Helpers
     /// </summary>
     public static class AuthenticationHelpers
     {
-        static ReaderWriterLockSlim _cacheLock = new ReaderWriterLockSlim(LockRecursionPolicy.SupportsRecursion);
+        static readonly ReaderWriterLockSlim _cacheLock = new ReaderWriterLockSlim(LockRecursionPolicy.SupportsRecursion);
 
         /// <summary>
         /// Signs out of the current session using the provided <see cref="IAuthContext"/>.
@@ -71,7 +71,6 @@ public static IAuthenticationProvider GetAuthProvider(IAuthContext authContext)
                         //https://github.com/AzureAD/microsoft-authentication-library-for-dotnet/wiki/MSAL.NET-uses-web-browser
                         IPublicClientApplication publicClientApp = PublicClientApplicationBuilder
                             .Create(authContext.ClientId)
-                            .WithTenantId(authContext.TenantId)
                             .WithAuthority(authorityUrl)
                             .WithClientCapabilities(new[] { "cp1" })
                             .WithDefaultRedirectUri()
@@ -98,7 +97,6 @@ public static IAuthenticationProvider GetAuthProvider(IAuthContext authContext)
                     {
                         IConfidentialClientApplication confidentialClientApp = ConfidentialClientApplicationBuilder
                             .Create(authContext.ClientId)
-                            .WithTenantId(authContext.TenantId)
                             .WithAuthority(authorityUrl)
                             .WithCertificate(GetCertificate(authContext))
                             .Build();