[ https://issues.apache.org/jira/browse/OAK-4930?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15712262#comment-15712262 ]
Manfred Baedke edited comment on OAK-4930 at 12/1/16 4:12 PM: -------------------------------------------------------------- Partially reverting [r1764678|http://svn.apache.org/r1764678]. Please see OAK-5200 for reasons. trunk: [r1772228|http://svn.apache.org/r1772228]. was (Author: baedke): Partially reverting [r1764678|http://svn.apache.org/r1764678]. Please see OAK-5200 for reasons. trunk: r1772228. > External Principal Management: DynamicSyncContext makes redundant calls to > IdentityProvider.getIdentity() > ---------------------------------------------------------------------------------------------------------- > > Key: OAK-4930 > URL: https://issues.apache.org/jira/browse/OAK-4930 > Project: Jackrabbit Oak > Issue Type: Improvement > Components: auth-external > Affects Versions: 1.4.8, 1.2.20, 1.5.12 > Reporter: Manfred Baedke > Assignee: Manfred Baedke > Fix For: 1.6, 1.4.9, 1.5.13 > > > When recursively collecting principal names associated with the declared > group memberships of a given principal, the method > DynamicSyncContext.collectPrincipalNames() unnecessarily calls > IdentityProvider.getIdentity() for every declared group reference, though it > is only necessary when the actual depth of the remaining recursion is >1. -- This message was sent by Atlassian JIRA (v6.3.4#6332)