[ 
https://issues.apache.org/jira/browse/OAK-709?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13621947#comment-13621947
 ] 

angela commented on OAK-709:
----------------------------

> Indeed, I don't think I claimed otherwise.

no, of course you didn't, but i wasn't sure if you see a simple way without
as usually try to avoid big refactoring as i don't like arguing :-)

> Note that the proposed SecureNodeState intentionally only covers read access 
> checks. 

sure! i didn't plan anything else....

> Write checks should remain as they are in the existing commit hook.

not that i would have suggested to change this. my point was that the
SecureNodeState needs to keep an immutable tree similar to the way it's
handled in the validators.
                
> Consider moving permission evaluation to the node state level
> -------------------------------------------------------------
>
>                 Key: OAK-709
>                 URL: https://issues.apache.org/jira/browse/OAK-709
>             Project: Jackrabbit Oak
>          Issue Type: Sub-task
>          Components: core
>            Reporter: angela
>         Attachments: 
> 0001-OAK-709-Consider-moving-permission-evaluation-to-the.patch, 
> 0001-OAK-709-Consider-moving-permission-evaluation-to-the.patch, 
> SecureNodeState.java
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to