[jira] [Commented] (SLING-6422) Allow for specifying oak restrictions with repoinit

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Commented] (SLING-6422) Allow for specifying oak restrictions with repoinit

JIRA jira@apache.org

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

Bertrand Delacretaz commented on SLING-6422:
--------------------------------------------

[~nitin.nizhawan] the {{sameRestrictions}} method assumes the {{getRestrictions}} and {{getMVRestrictions}} methods return a sorted list of values - in the for loop that compares old and new values.

Is that ordering guaranteed? The javadocs [1] do not mention ordering.

If the Value array is not sorted I think you need a more robust comparison.

[1] https://jackrabbit.apache.org/api/2.8/org/apache/jackrabbit/api/security/JackrabbitAccessControlEntry.html#getRestrictions(java.lang.String)

> Allow for specifying oak restrictions with repoinit
> ---------------------------------------------------
>
>                 Key: SLING-6422
>                 URL: https://issues.apache.org/jira/browse/SLING-6422
>             Project: Sling
>          Issue Type: New Feature
>          Components: Repoinit
>            Reporter: Nitin Nizhawan
>         Attachments: SLING6422ApplyRestrictionsV2.patch, SLING6422ApplyRestrictionsV3.patch, SLING6422_interpretparsedrestrictionclause.patch, SLING-6422.patch
>
>
> Allow for specifying oak restrictions with repoinit. Currently repoinit allows one to ADD remove ACLs but there is no way to specify oak restrictions.
> http://jackrabbit.apache.org/oak/docs/security/authorization/restriction.html



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)