[jira] [Commented] (SLING-8619) RepoInitGrammer: Add repository-level marker to pathsList

["angela (JIRA)" <ji...@apache.org>]

    [ 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)