[ https://issues.apache.org/jira/browse/JCR-4249?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16331333#comment-16331333 ]
angela commented on JCR-4249: ----------------------------- [~stillalex], the patch looks good to me... the only thing that we might want to add: I would mention in the javadoc of {{PrincipalManager}} that it used to be {{java.security.acl.Group}} and when + why this has been adjusted... as long as it's not remove implementations might have prinicipals that implement both interfaces for backwards compatibility. wdyt? > Introduce replacement for java.security.acl.Group > ------------------------------------------------- > > Key: JCR-4249 > URL: https://issues.apache.org/jira/browse/JCR-4249 > Project: Jackrabbit Content Repository > Issue Type: Improvement > Components: JCR API, security > Reporter: Alex Deparvu > Assignee: Alex Deparvu > Priority: Major > Fix For: 2.17.1 > > Attachments: JCR-4249.patch > > > See OAK-7024 and JCR-4246 for more information. > I've decided to split off the api changes to have them in early for a release > so both projects could use the same class as replacement of the Group. -- This message was sent by Atlassian JIRA (v7.6.3#76005)