[GitHub] [pulsar] merlimat opened a new pull request #12064: [PIP-45] Remove ConfigurationCacheService from AuthorizationProvider

[GitBox <gi...@apache.org>]

merlimat opened a new pull request #12064:
URL: https://github.com/apache/pulsar/pull/12064


   ### Motivation
   
   The `AuthorizationProvider` is initialized by passing a reference to the `ConfigurationCacheService`. 
   
   `ConfigurationCacheService` is completely tied to ZK access and to the ZoookeeperCache classes and it has been superseded by `PulsarResources`. As of now, the default authorization implementation is just using the `ConfigurationCacheService` to pull out a ref to the `PulsarResources`. 
   
   We need to remove `ConfigurationCacheService` from the `AuthorizationProvider` and pass instead the `PulsarResources`, while at the same time maintaining some degree of compatibility in the interface.
   
   The rationale here is that, this reference is in practice only needed in the Pulsar default authz implementation (where the ACLs are stored in ZK). 3rd party implementations will be validating the ACLs through other means, thus not accessing the internal ZK service.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar] merlimat merged pull request #12064: [PIP-45] Remove ConfigurationCacheService from AuthorizationProvider

[GitBox <gi...@apache.org>]

merlimat merged pull request #12064:
URL: https://github.com/apache/pulsar/pull/12064


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org