You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2020/03/09 13:57:52 UTC

[GitHub] [pulsar] KannarFr commented on issue #6428: [Issue 5720][authorization provider] (WIP) Add more granularity

KannarFr commented on issue #6428: [Issue 5720][authorization provider] (WIP) Add more granularity
URL: https://github.com/apache/pulsar/pull/6428#issuecomment-596539275
 
 
   @sijie operations like `internalGrantPermissionOnNamespace` in `NamespacesBase.java` are antipattern with the reworking I'm doing. I don't know how to manage them. I think I should let them as they are. Due to "real" authz plugin we should drop all roles/actions stuff which are in ZK. DefaultAuthzProvider should be updated regarding this.
   
   That being said, we need maintain legacy stuff so I would like to have your opinions about this.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services