Automatically set LoadMetadata=true if MetadataLocation is present in…

… web.config - Fixes #675
Sustainsys · Mar 29, 2017 · a7e58b5 · a7e58b5
1 parent c4b05a1
commit a7e58b5
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/Kentor.AuthServices.Tests/IdentityProviderTests.cs b/Kentor.AuthServices.Tests/IdentityProviderTests.cs
@@ -416,7 +416,6 @@ public void IdentityProvider_Ctor_PrefersRedirectBindingForLogout()
         public void IdentityProvider_Ctor_UseMetadataLocationUrl()
         {
             var config = CreateConfig();
-            config.LoadMetadata = true;
             config.MetadataLocation = "http://localhost:13428/idpMetadataDifferentEntityId";
             config.EntityId = "some-idp";
 

diff --git a/Kentor.AuthServices/IdentityProvider.cs b/Kentor.AuthServices/IdentityProvider.cs
@@ -76,7 +76,7 @@ internal IdentityProvider(IdentityProviderElement config, SPOptions spOptions)
 
             // If configured to load metadata, this will immediately do the load.
             this.spOptions = spOptions;
-            LoadMetadata = config.LoadMetadata;
+            LoadMetadata = config.LoadMetadata || !string.IsNullOrEmpty(config.MetadataLocation);
 
             // Validate if values are only from config. If metadata is loaded, validation
             // is done on metadata load.