commit 9e593b31f902128755dbb902c26577489e6c69a5
author Bertrand Delacretaz <bdelacretaz@apache.org> Fri Feb 05 12:06:31 2016 +0000
committer Bertrand Delacretaz <bdelacretaz@apache.org> Fri Feb 05 12:06:31 2016 +0000
tree 4590a9311f15402d11a1e0ee95bd785107fb2882
parent ad47cb47b67fb5ae706697c5d65e9d3c6c72f30b

SLING-5484 - tweak debug logs

git-svn-id: https://svn.apache.org/repos/asf/sling/trunk@1728647 13f79535-47bb-0310-9956-ffa450edef68

src/main/java/org/apache/sling/serviceusermapping/impl/ServiceUserMapperImpl.java

diff --git a/src/main/java/org/apache/sling/serviceusermapping/impl/ServiceUserMapperImpl.java b/src/main/java/org/apache/sling/serviceusermapping/impl/ServiceUserMapperImpl.java
index edd22dd..d4bc867 100644
--- a/src/main/java/org/apache/sling/serviceusermapping/impl/ServiceUserMapperImpl.java
+++ b/src/main/java/org/apache/sling/serviceusermapping/impl/ServiceUserMapperImpl.java
@@ -240,7 +240,7 @@
         }
 
         activeMappings = mappings.toArray(new Mapping[mappings.size()]);
-        log.debug("Active mappings updated: {}", mappings);
+        log.debug("Active mappings updated: {} mappings active", mappings.size());
 
         RegistrationSet registrationSet = updateServiceRegistrations(activeMappings);
 
@@ -358,7 +358,10 @@
     }
 
     private String internalGetUserId(final String serviceName, final String subServiceName) {
-        // try with serviceInfo first
+        log.debug(
+                "internalGetUserId: {} active mappings, looking for mapping for {}/{}", 
+                new Object[] { this.activeMappings.length, serviceName, subServiceName });
+        
         for (final Mapping mapping : this.activeMappings) {
             final String userId = mapping.map(serviceName, subServiceName);
             if (userId != null) {
@@ -368,6 +371,10 @@
         }
 
         // second round without serviceInfo
+        log.debug(
+                "internalGetUserId: {} active mappings, looking for mapping for {}/<no subServiceName>", 
+                this.activeMappings.length, serviceName);
+        
         for (Mapping mapping : this.activeMappings) {
             final String userId = mapping.map(serviceName, null);
             if (userId != null) {
@@ -376,7 +383,7 @@
             }
         }
 
-        log.debug("No mapping found, fallback to default user [{}]", this.defaultUser);
+        log.debug("internalGetUserId: no mapping found, fallback to default user [{}]", this.defaultUser);
         return this.defaultUser;
     }