[java-identity-provider] 06/06: IDP-1121 Remove never-present mergeResult LDAP attribute
Rod Widdowson
rdw at steadingsoftware.com
Wed Apr 3 02:29:50 EDT 2019
This is an automated email from the git hooks/post-receive script.
rdw pushed a commit to branch master
in repository java-identity-provider.
View the commit online:
http://git.shibboleth.net/view/?p=java-identity-provider.git;a=commit;h=25a7c01eaf254ece3043c90995e6e331f6804ba8
commit 25a7c01eaf254ece3043c90995e6e331f6804ba8
Author: Rod Widdowson <rdw at steadingsoftware.com>
AuthorDate: Tue Apr 2 16:46:02 2019 +0100
IDP-1121 Remove never-present mergeResult LDAP attribute
https://issues.shibboleth.net/jira/browse/IDP-1121
---
.../resolver/spring/dc/ldap/impl/CredentialConfigFactoryBean.java | 6 ++----
.../resolver/spring/dc/ldap/impl/LDAPDataConnectorParser.java | 6 ------
2 files changed, 2 insertions(+), 10 deletions(-)
diff --git a/idp-attribute-resolver-spring/src/main/java/net/shibboleth/idp/attribute/resolver/spring/dc/ldap/impl/CredentialConfigFactoryBean.java b/idp-attribute-resolver-spring/src/main/java/net/shibboleth/idp/attribute/resolver/spring/dc/ldap/impl/CredentialConfigFactoryBean.java
index 32065a6..e92d9b4 100644
--- a/idp-attribute-resolver-spring/src/main/java/net/shibboleth/idp/attribute/resolver/spring/dc/ldap/impl/CredentialConfigFactoryBean.java
+++ b/idp-attribute-resolver-spring/src/main/java/net/shibboleth/idp/attribute/resolver/spring/dc/ldap/impl/CredentialConfigFactoryBean.java
@@ -23,10 +23,6 @@ import java.security.cert.X509Certificate;
import javax.annotation.Nonnull;
import javax.annotation.Nullable;
-import net.shibboleth.ext.spring.factory.AbstractComponentAwareFactoryBean;
-import net.shibboleth.utilities.java.support.primitive.DeprecationSupport;
-import net.shibboleth.utilities.java.support.primitive.DeprecationSupport.ObjectType;
-
import org.ldaptive.ssl.CredentialConfig;
import org.ldaptive.ssl.CredentialConfigFactory;
import org.opensaml.security.credential.Credential;
@@ -35,6 +31,8 @@ import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import org.springframework.beans.factory.BeanCreationException;
+import net.shibboleth.ext.spring.factory.AbstractComponentAwareFactoryBean;
+
/**
* A Factory bean to summon up {@link CredentialConfig} from supplied <Credential> statements.
*/
diff --git a/idp-attribute-resolver-spring/src/main/java/net/shibboleth/idp/attribute/resolver/spring/dc/ldap/impl/LDAPDataConnectorParser.java b/idp-attribute-resolver-spring/src/main/java/net/shibboleth/idp/attribute/resolver/spring/dc/ldap/impl/LDAPDataConnectorParser.java
index 1c40a39..4c467cb 100644
--- a/idp-attribute-resolver-spring/src/main/java/net/shibboleth/idp/attribute/resolver/spring/dc/ldap/impl/LDAPDataConnectorParser.java
+++ b/idp-attribute-resolver-spring/src/main/java/net/shibboleth/idp/attribute/resolver/spring/dc/ldap/impl/LDAPDataConnectorParser.java
@@ -65,8 +65,6 @@ import net.shibboleth.idp.attribute.resolver.spring.dc.impl.CacheConfigParser;
import net.shibboleth.idp.attribute.resolver.spring.impl.AttributeResolverNamespaceHandler;
import net.shibboleth.idp.profile.spring.factory.BasicX509CredentialFactoryBean;
import net.shibboleth.utilities.java.support.logic.Constraint;
-import net.shibboleth.utilities.java.support.primitive.DeprecationSupport;
-import net.shibboleth.utilities.java.support.primitive.DeprecationSupport.ObjectType;
import net.shibboleth.utilities.java.support.primitive.StringSupport;
import net.shibboleth.utilities.java.support.xml.AttributeSupport;
import net.shibboleth.utilities.java.support.xml.ElementSupport;
@@ -249,10 +247,6 @@ public class LDAPDataConnectorParser extends AbstractDataConnectorParser {
Constraint.isNotNull(config, "LDAPDirectory element cannot be null");
configElement = config;
logPrefix = prefix;
- // warn about deprecated (removed?) attribute
- if (AttributeSupport.hasAttribute(config, new QName("mergeResults"))) {
- DeprecationSupport.warn(ObjectType.ATTRIBUTE, "mergeResults", prefix, null);
- }
}
/**
--
To stop receiving notification emails like this one, please contact
the administrator of this repository.
More information about the commits
mailing list