diff --git a/backend/src/main/groovy/edu/internet2/tier/shibboleth/admin/ui/service/JPAMetadataResolverServiceImpl.groovy b/backend/src/main/groovy/edu/internet2/tier/shibboleth/admin/ui/service/JPAMetadataResolverServiceImpl.groovy index 4c0229b22..049869b64 100644 --- a/backend/src/main/groovy/edu/internet2/tier/shibboleth/admin/ui/service/JPAMetadataResolverServiceImpl.groovy +++ b/backend/src/main/groovy/edu/internet2/tier/shibboleth/admin/ui/service/JPAMetadataResolverServiceImpl.groovy @@ -32,6 +32,7 @@ import groovy.util.logging.Slf4j import groovy.xml.DOMBuilder import groovy.xml.MarkupBuilder import net.shibboleth.utilities.java.support.scripting.EvaluableScript +import org.apache.commons.collections.CollectionUtils import org.opensaml.saml.common.profile.logic.EntityIdPredicate import org.opensaml.saml.metadata.resolver.MetadataResolver import org.opensaml.saml.metadata.resolver.filter.MetadataFilter @@ -229,7 +230,7 @@ class JPAMetadataResolverServiceImpl implements MetadataResolverService { maxConnectionsTotal: resolver.maxConnectionsTotal, maxConnectionsPerRoute: resolver.maxConnectionsPerRoute, - supportedContentTypes: resolver.supportedContentTypes?.value, //not sure this is right. maybe take off the ?.value + supportedContentTypes: CollectionUtils.isEmpty(resolver.supportedContentTypes ) ? null : resolver.supportedContentTypes, httpClientRef: resolver.httpMetadataResolverAttributes?.httpClientRef, connectionRequestTimeout: resolver.httpMetadataResolverAttributes?.connectionRequestTimeout, @@ -436,7 +437,7 @@ class JPAMetadataResolverServiceImpl implements MetadataResolverService { xml.omitEmptyAttributes = true xml.omitNullAttributes = true - xml.MetadataProvider(id: 'ShibbolethMetadata', + xml.MetadataProvider(id: 'ShibbolethIdPUIGeneratedMetadata', xmlns: 'urn:mace:shibboleth:2.0:metadata', 'xmlns:xsi': 'http://www.w3.org/2001/XMLSchema-instance', 'xsi:type': 'ChainingMetadataProvider', diff --git a/backend/src/test/resources/conf/1059-disabled.xml b/backend/src/test/resources/conf/1059-disabled.xml index 952c86ee6..c53ed9bb1 100644 --- a/backend/src/test/resources/conf/1059-disabled.xml +++ b/backend/src/test/resources/conf/1059-disabled.xml @@ -1,6 +1,6 @@ - - - + + diff --git a/backend/src/test/resources/conf/278.xml b/backend/src/test/resources/conf/278.xml index 46e57a6b4..918ffc0ae 100644 --- a/backend/src/test/resources/conf/278.xml +++ b/backend/src/test/resources/conf/278.xml @@ -1,5 +1,5 @@ - - + + diff --git a/backend/src/test/resources/conf/670.xml b/backend/src/test/resources/conf/670.xml index 952c86ee6..c53ed9bb1 100644 --- a/backend/src/test/resources/conf/670.xml +++ b/backend/src/test/resources/conf/670.xml @@ -1,6 +1,6 @@ - + diff --git a/backend/src/test/resources/conf/984.xml b/backend/src/test/resources/conf/984.xml index 42328a0c0..ad76d7bbd 100644 --- a/backend/src/test/resources/conf/984.xml +++ b/backend/src/test/resources/conf/984.xml @@ -1,4 +1,4 @@ - +