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

Reply via email to