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

Jukka Zitting commented on OAK-709:
-----------------------------------

bq. so, it will include major refactoring 

Indeed, I don't think I claimed otherwise.

bq. SecureNodeStore (in a first step for each content session) that holds the 
permissionprovider

I don't see a need for wrapping NodeStore, as we can put the relevant code 
directly in RootImpl.

Note that the proposed SecureNodeState intentionally only covers read access 
checks. Write checks should remain as they are in the existing commit hook.
                
> 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, 
> 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