[OAK-4218] Base SyncMBeanImpl on Oak API

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.5.4, 1.6.0
• Component/s: auth-external
• Assignee: Angela Schreiber
• Reporter: Angela Schreiber
• Created: 18/Apr/16 12:16
• Updated: 10/Nov/16 09:47
• Resolved: 15/Jun/16 16:07

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.

Issue Links

• relates to: OAK-4101 Consider separate external (group) principal management
• requires: OAK-4264 Improve testing of SyncMBeanImpl