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

Marcel Reutegger commented on OAK-7756:
---------------------------------------

Registering a {{NodeStore}} also as an {{Observable}} sounds reasonable to me, 
but since the {{Observable}} provides read access to the {{NodeStore}}, I would 
think the same application code that restricts access to the {{NodeStore}} will 
also restrict access to an {{Observable}}.

So even if Oak exposed the {{NodeStore}} as an {{Observable}} you'd be back to 
square one.

> Node stores are not registered as Observable
> --------------------------------------------
>
>                 Key: OAK-7756
>                 URL: https://issues.apache.org/jira/browse/OAK-7756
>             Project: Jackrabbit Oak
>          Issue Type: Bug
>            Reporter: Oliver Lietz
>            Priority: Major
>
> The recently released [Sling Clam 
> module|https://github.com/apache/sling-org-apache-sling-clam] uses the 
> NodeStore service for listening to add and change events.
> That works fine on Sling but a cast to {{Observable}} is required.
> On AEM it works not OOTB as 
> {{com.day.crx.sling.server.impl.jmx.SecureContentRepositoryAccess}} prevents 
> direct access to {{NodeStore}} and therefore 
> {{SecureContentRepositoryAccess}} component has to be disabled to make it 
> work.
> To get rid of the cast and the limitation on AEM node stores should be 
> registered as {{Observable}} also.
>  * {{SegmentNodeStore}} in {{SegmentNodeStoreService}}
>  * {{DocumentNodeStore}} in {{DocumentNodeStoreService}}
>  * {{CompositeNodeStore}} in {{CompositeNodeStoreService}}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to