take one client secret from options (#577)

AzureAD · Sep 15, 2020 · 0580848 · 0580848
1 parent e97d011
commit 0580848
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/src/Microsoft.Identity.Web/TokenAcquisition.cs b/src/Microsoft.Identity.Web/TokenAcquisition.cs
@@ -457,9 +457,14 @@ private async Task<IConfidentialClientApplication> BuildConfidentialClientApplic
                 _applicationOptions.Instance += "/";
             }
 
+            if (!string.IsNullOrEmpty(_microsoftIdentityOptions.ClientSecret))
+            {
+                _applicationOptions.ClientSecret = _microsoftIdentityOptions.ClientSecret;
+            }
+
             MicrosoftIdentityOptionsValidation.ValidateEitherClientCertificateOrClientSecret(
-                _applicationOptions.ClientSecret,
-                _microsoftIdentityOptions.ClientCertificates);
+                 _applicationOptions.ClientSecret,
+                 _microsoftIdentityOptions.ClientCertificates);
 
             try
             {

diff --git a/tests/IntegrationTests/IntegrationTestService/Startup.cs b/tests/IntegrationTests/IntegrationTestService/Startup.cs
@@ -42,10 +42,6 @@ public void ConfigureServices(IServiceCollection services)
             {
                 options.ClientSecret = ccaSecret;
             });
-            services.Configure<ConfidentialClientApplicationOptions>(options =>
-            {
-                options.ClientSecret = ccaSecret;
-            });
             services.AddRazorPages(options =>
             {
                 options.Conventions.AuthorizePage("/SecurePage");