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

Base SyncMBeanImpl on Oak API

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

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 1.5.4, 1.6.0
    • auth-external
    • None

    Description

      While looking at the oak-auth-external code base I found that SyncMBeanImpl is based on JCR API while sync called during authentication will solely rely on API defined by oak-core.

      This not only limits the implementations to operations that can executed on the JCR API but also introduces the risk of inconsistencies.

      As a matter of fact ExternalLoginModuleTestBase.createMBean also lists this as limitation and TODO:

      // todo: how to retrieve JCR repository here? maybe we should base the sync mbean on oak directly.

      Attachments

        Issue Links

        relates to

        New Feature - A new feature of the product, which has yet to be developed. OAK-4101 Consider separate external (group) principal management

        • Major - Major loss of function.
        • Closed
        requires

        Test - A new unit, integration or system test. OAK-4264 Improve testing of SyncMBeanImpl

        • Major - Major loss of function.
        • Closed

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

          People

            angela Angela Schreiber
            angela Angela Schreiber
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment