You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-issues@jackrabbit.apache.org by "angela (JIRA)" <ji...@apache.org> on 2016/11/30 17:41:58 UTC

[jira] [Commented] (OAK-4930) External Principal Management: DynamicSyncContext makes redundant calls to IdentityProvider.getIdentity()

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

angela commented on OAK-4930:
-----------------------------

[~baedke], changed back to Improvement as discussed on the phone... I really don't think this was a bug.

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