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

[noreply at shibboleth.net]

Author: rdw
Date: Wed Apr  3 10:17:48 2013
New Revision: 4386

URL: http://svn.shibboleth.net/view/java-identity-provider?rev=4386&view=rev
Log:
Clean up logging slightly

Modified:
    trunk/idp-attribute-resolver-impl/src/main/java/net/shibboleth/idp/attribute/resolver/impl/ad/ScriptedAttributeDefinition.java

Modified: trunk/idp-attribute-resolver-impl/src/main/java/net/shibboleth/idp/attribute/resolver/impl/ad/ScriptedAttributeDefinition.java
URL: http://svn.shibboleth.net/view/java-identity-provider/trunk/idp-attribute-resolver-impl/src/main/java/net/shibboleth/idp/attribute/resolver/impl/ad/ScriptedAttributeDefinition.java?rev=4386&r1=4385&r2=4386&view=diff
==============================================================================
--- trunk/idp-attribute-resolver-impl/src/main/java/net/shibboleth/idp/attribute/resolver/impl/ad/ScriptedAttributeDefinition.java (original)
+++ trunk/idp-attribute-resolver-impl/src/main/java/net/shibboleth/idp/attribute/resolver/impl/ad/ScriptedAttributeDefinition.java Wed Apr  3 10:17:48 2013
@@ -106,12 +106,12 @@
         try {
             script.eval(context);
         } catch (ScriptException e) {
-            throw new ResolutionException("AttributeDefinition '" + getId() + "' unable to execute script", e);
+            throw new ResolutionException("AttributeDefinition '" + getId() + "': unable to execute script", e);
         }
         Object result = context.getAttribute(getId());
 
         if (null == result) {
-            log.info("AttributeDefinition '{}' No value returned", getId());
+            log.info("AttributeDefinition '{}': No value returned", getId());
             return Optional.absent();
         }
 
@@ -122,7 +122,7 @@
 
         } else {
 
-            throw new ResolutionException("AttributeDefinition '" + getId() + "'returned variable was of wrong type ("
+            throw new ResolutionException("AttributeDefinition '" + getId() + "': returned variable was of wrong type ("
                     + result.getClass().toString() + ")");
         }