Uploaded image for project: 'Jackrabbit Oak'
  1. Jackrabbit Oak
  2. OAK-4930

External Principal Management: DynamicSyncContext makes redundant calls to IdentityProvider.getIdentity()

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 1.2.20, 1.4.8, 1.5.12
    • 1.4.9, 1.5.13, 1.6.0
    • auth-external
    • None

    Description

      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.

      Attachments

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            baedke Manfred Baedke
            baedke Manfred Baedke
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment