You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by kw...@apache.org on 2017/03/30 10:46:20 UTC

svn commit: r1789494 - /sling/trunk/bundles/jcr/jackrabbit-usermanager/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/resource/AuthorizableValueMap.java

Author: kwin
Date: Thu Mar 30 10:46:20 2017
New Revision: 1789494

URL: http://svn.apache.org/viewvc?rev=1789494&view=rev
Log:
add more logging

Modified:
    sling/trunk/bundles/jcr/jackrabbit-usermanager/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/resource/AuthorizableValueMap.java

Modified: sling/trunk/bundles/jcr/jackrabbit-usermanager/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/resource/AuthorizableValueMap.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/jcr/jackrabbit-usermanager/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/resource/AuthorizableValueMap.java?rev=1789494&r1=1789493&r2=1789494&view=diff
==============================================================================
--- sling/trunk/bundles/jcr/jackrabbit-usermanager/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/resource/AuthorizableValueMap.java (original)
+++ sling/trunk/bundles/jcr/jackrabbit-usermanager/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/resource/AuthorizableValueMap.java Thu Mar 30 10:46:20 2017
@@ -229,6 +229,7 @@ public class AuthorizableValueMap implem
                 cache.put(MEMBER_OF_KEY, getMemberships(authorizable, true));
                 cache.put(DECLARED_MEMBER_OF_KEY, getMemberships(authorizable, false));
 
+                // only direct properties are supported here
                 Iterator<String> pi = authorizable.getPropertyNames();
                 while (pi.hasNext()) {
                     String key = (String) pi.next();
@@ -241,7 +242,7 @@ public class AuthorizableValueMap implem
 
                 fullyRead = true;
             } catch (RepositoryException re) {
-                // TODO: log !!
+                LOG.error("Could not access certain properties of user {}", authorizable, re);
             }
         }
     }