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

Reply via email to