[java-identity-provider COMMIT] /trunk/idp-attribute-resolver-spring/src/main/java/net/shibboleth/idp/attribute/resol...

[noreply at shibboleth.net]

Author: rdw
Date: Mon Dec  7 10:16:54 2015
New Revision: 8018

URL: http://svn.shibboleth.net/view/java-identity-provider?rev=8018&view=rev
Log:
IDP-877  Fail in parsing if dependencies not supplied correctly.

https://issues.shibboleth.net/jira/browse/IDP-877

Revert last.  "Its not that easy"

Modified:
    trunk/idp-attribute-resolver-spring/src/main/java/net/shibboleth/idp/attribute/resolver/spring/ad/BaseAttributeDefinitionParser.java

Modified: trunk/idp-attribute-resolver-spring/src/main/java/net/shibboleth/idp/attribute/resolver/spring/ad/BaseAttributeDefinitionParser.java
URL: http://svn.shibboleth.net/view/java-identity-provider/trunk/idp-attribute-resolver-spring/src/main/java/net/shibboleth/idp/attribute/resolver/spring/ad/BaseAttributeDefinitionParser.java?rev=8018&r1=8017&r2=8018&view=diff
==============================================================================
--- trunk/idp-attribute-resolver-spring/src/main/java/net/shibboleth/idp/attribute/resolver/spring/ad/BaseAttributeDefinitionParser.java	(original)
+++ trunk/idp-attribute-resolver-spring/src/main/java/net/shibboleth/idp/attribute/resolver/spring/ad/BaseAttributeDefinitionParser.java	Mon Dec  7 10:16:54 2015
@@ -34,7 +34,6 @@
 
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
-import org.springframework.beans.factory.BeanCreationException;
 import org.springframework.beans.factory.support.BeanDefinitionBuilder;
 import org.springframework.beans.factory.support.ManagedMap;
 import org.springframework.beans.factory.xml.ParserContext;
@@ -98,8 +97,7 @@
                         getLogPrefix(), sourceAttributeId);
             }
         } else if (needsAttributeSourceID()) {
-            log.error("{} sourceAttributeID was not specified but is required", getLogPrefix());
-            throw new BeanCreationException(getLogPrefix()+": sourceAttributeID was not specified but is required");
+            log.warn("{} sourceAttributeID was not specified but is required", getLogPrefix());
         }
 
         final List<Element> attributeEncoders =