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

angela commented on OAK-7203:
-----------------------------

[~stillalex], [~rombert], I just want to highlight the fact that the 
{{CugConfiguration}} also contains the same reference to {{MountInfoProvider}}. 
in fact the unbind method even contains the following comment, which leaves me 
with the impression that having the references was initially planned to be 
mandatory:

{code}
// set to null (and not default) to comply with OSGi lifecycle,
// if the reference is unset it means the service is being deactivated
{code}

I don't have any preference wrt the exact nature of the {{MountInfoProvider}} 
within the authorization setup, but I think it must be consistent across the 2 
implementations we ship (adding authorization-cug component as well.

> Make MountInfoProvider service in AuthorizationConfigurationImpl optional
> -------------------------------------------------------------------------
>
>                 Key: OAK-7203
>                 URL: https://issues.apache.org/jira/browse/OAK-7203
>             Project: Jackrabbit Oak
>          Issue Type: Improvement
>          Components: authorization-cug, core
>    Affects Versions: 1.8.1
>            Reporter: Oliver Lietz
>            Priority: Major
>         Attachments: OAK-7203.patch
>
>
> While testing Sling with Oak 1.8 I've observed that 
> AuthorizationConfigurationImpl gets not activated due to missingĀ 
> MountInfoProvider service:
> {noformat}
>     @Reference
>     private MountInfoProvider mountInfoProvider = 
> Mounts.defaultMountInfoProvider();
> {noformat}
> {noformat}
> org.apache.jackrabbit.oak.security.authorization.AuthorizationConfigurationImpl
> Bundle        org.apache.jackrabbit.oak-core (63)
> Implementation Class  
> org.apache.jackrabbit.oak.security.authorization.AuthorizationConfigurationImpl
> Default State enabled
> Activation    delayed
> Configuration Policy  optional
> Service Type  singleton
> Services      
> org.apache.jackrabbit.oak.spi.security.authorization.AuthorizationConfiguration
> org.apache.jackrabbit.oak.spi.security.SecurityConfiguration
> PID   
> org.apache.jackrabbit.oak.security.authorization.AuthorizationConfigurationImpl
> Reference mountInfoProvider   Unsatisfied
> Service Name: org.apache.jackrabbit.oak.spi.mount.MountInfoProvider
> Cardinality: 1..1
> Policy: static
> Policy Option: reluctant
> No Services bound
> Properties    component.id = 35
> component.name = 
> org.apache.jackrabbit.oak.security.authorization.AuthorizationConfigurationImpl
> configurationRanking = 100
> importBehavior = abort
> oak.security.name = 
> org.apache.jackrabbit.oak.security.authorization.AuthorizationConfigurationImpl
> readPaths = [/jcr:system/rep:namespaces, /jcr:system/jcr:nodeTypes, 
> /jcr:system/rep:privileges]
> {noformat}



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

Reply via email to