[
https://issues.apache.org/jira/browse/OAK-1147?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13814015#comment-13814015
]
Jukka Zitting commented on OAK-1147:
------------------------------------
bq. Jukka Zitting, do you remember what was the reason for using #canReadAll
here?
I think I was just duplicating the pattern from child node access methods,
without realizing that we also had the {{canReadAllProperties()}} check that's
more appropriate for these methods.
> 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)
