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 "Marcel Reutegger (JIRA)" <ji...@apache.org> on 2018/09/17 13:00:00 UTC

[jira] [Commented] (OAK-7756) Node stores are not registered as Observable

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