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

angela commented on SLING-8619:
-------------------------------

[~rombert], [~karlpauls], you have been recently working on the repo init code 
base. i would appreciate if you could review the 2 patches for 
_sling-repoinit-parser_ and _sling-jcr-repoinit_ respectively.

note: the latter obviously needs to have the snapshot dependency to the parser 
replaced once the former is released. i just added it in order to have a patch 
that can be successfully build with passing tests.

one more comment: i would have wished to include an update to the documentation 
along with the patches but didn't find it with either module... am i missing 
something?

> RepoInitGrammer: Add repository-level marker to pathsList
> ---------------------------------------------------------
>
>                 Key: SLING-8619
>                 URL: https://issues.apache.org/jira/browse/SLING-8619
>             Project: Sling
>          Issue Type: Improvement
>          Components: Repoinit
>            Reporter: angela
>            Priority: Major
>         Attachments: SLING-8619-jcr.patch, SLING-8619-parser.patch
>
>
> In order to be able to cover repository level access control entrieswith the 
> {{PrincipalAccessControlList}} (see SLING-8602), i would like to expand the 
> {{RepoInitGrammer.pathsList()}} such that it not only handles absolute paths 
> but in addition a marker for the repository level permissions (in JCR marked 
> with a _null_ path).
> essentially this would also create an alternative way to edit policies for 
> the _null_ path (today covered with _set repository ACL ..._).
> will provide a proposed patch and tests later this week.



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)

Reply via email to