Author: olamy
Date: Fri Mar  1 23:44:13 2013
New Revision: 1451780

URL: http://svn.apache.org/r1451780
Log:
better logging

Modified:
    
archiva/redback/redback-core/trunk/redback-common/redback-common-ldap/src/main/java/org/apache/archiva/redback/common/ldap/role/DefaultLdapRoleMapperConfiguration.java

Modified: 
archiva/redback/redback-core/trunk/redback-common/redback-common-ldap/src/main/java/org/apache/archiva/redback/common/ldap/role/DefaultLdapRoleMapperConfiguration.java
URL: 
http://svn.apache.org/viewvc/archiva/redback/redback-core/trunk/redback-common/redback-common-ldap/src/main/java/org/apache/archiva/redback/common/ldap/role/DefaultLdapRoleMapperConfiguration.java?rev=1451780&r1=1451779&r2=1451780&view=diff
==============================================================================
--- 
archiva/redback/redback-core/trunk/redback-common/redback-common-ldap/src/main/java/org/apache/archiva/redback/common/ldap/role/DefaultLdapRoleMapperConfiguration.java
 (original)
+++ 
archiva/redback/redback-core/trunk/redback-common/redback-common-ldap/src/main/java/org/apache/archiva/redback/common/ldap/role/DefaultLdapRoleMapperConfiguration.java
 Fri Mar  1 23:44:13 2013
@@ -41,7 +41,7 @@ import java.util.Map;
  * @author Olivier Lamy
  * @since 2.1
  */
-@Service( "ldapRoleMapperConfiguration#default" )
+@Service("ldapRoleMapperConfiguration#default")
 public class DefaultLdapRoleMapperConfiguration
     implements LdapRoleMapperConfiguration
 {
@@ -51,14 +51,14 @@ public class DefaultLdapRoleMapperConfig
     private Map<String, List<String>> ldapMappings = new HashMap<String, 
List<String>>();
 
     @Inject
-    @Named( value = "userConfiguration#default" )
+    @Named(value = "userConfiguration#default")
     private UserConfiguration userConf;
 
     public void addLdapMapping( String ldapGroup, List<String> roles )
         throws MappingException
     {
         ldapMappings.put( ldapGroup, roles );
-        log.warn( "addLdapMapping implemented but only in memory save" );
+        log.warn( "addLdapMapping implemented but only in memory save: group 
'{}' roles '{}'", ldapGroup, roles );
     }
 
     public void removeLdapMapping( String group )
@@ -70,7 +70,7 @@ public class DefaultLdapRoleMapperConfig
         throws MappingException
     {
         ldapMappings.put( ldapGroup, roles );
-        log.warn( "updateLdapMapping implemented but only in memory save" );
+        log.warn( "updateLdapMapping implemented but only in memory save: 
group '{}' roles '{}'", ldapGroup, roles );
     }
 
     public void setLdapGroupMappings( Map<String, List<String>> mappings )


Reply via email to