From a8f651e8a638eb043bd9a2e8f2fd8916a7623efe Mon Sep 17 00:00:00 2001
From: Olivier Lamy <olamy@apache.org>
Date: Fri, 1 Mar 2013 23:44:13 +0000
Subject: [PATCH] better logging

git-svn-id: https://svn.apache.org/repos/asf/archiva/redback/redback-core/trunk@1451780 13f79535-47bb-0310-9956-ffa450edef68
---
 .../ldap/role/DefaultLdapRoleMapperConfiguration.java     | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/redback-common/redback-common-ldap/src/main/java/org/apache/archiva/redback/common/ldap/role/DefaultLdapRoleMapperConfiguration.java b/redback-common/redback-common-ldap/src/main/java/org/apache/archiva/redback/common/ldap/role/DefaultLdapRoleMapperConfiguration.java
index 5659e25e..95201ac2 100644
--- a/redback-common/redback-common-ldap/src/main/java/org/apache/archiva/redback/common/ldap/role/DefaultLdapRoleMapperConfiguration.java
+++ b/redback-common/redback-common-ldap/src/main/java/org/apache/archiva/redback/common/ldap/role/DefaultLdapRoleMapperConfiguration.java
@@ -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 DefaultLdapRoleMapperConfiguration
     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 DefaultLdapRoleMapperConfiguration
         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 )