angela created OAK-1147:
---------------------------

             Summary: SecureNodeBuilder/SecureNodeState: Consider using 
'TreePermission#canReadProperties'
                 Key: OAK-1147
                 URL: https://issues.apache.org/jira/browse/OAK-1147
             Project: Jackrabbit Oak
          Issue Type: Improvement
            Reporter: angela
            Assignee: angela


the methods #getProperties and #getPropertyCount have a shortcut for those 
cases where all properties are accessible.

however, the current implemention requires SecurityContext#canReadAll to return 
true in order to enable the shortcut. without knowing the very details of the 
SecureNodeState/Builder i would have expected that #canReadAllProperties would 
be sufficient.

[~jukkaz], do you remember what was the reason for using #canReadAll here? i 
changed it for test purpose and didn't see any difference... but that may also 
imply that we don't have enough or specific tests for this.



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Reply via email to