yuruguo opened a new pull request #12742:
URL: https://github.com/apache/pulsar/pull/12742


   ### Motivation
   Currently, we can `unsubscribe` the given subscription on all topics on a 
namespace through `bin/pulsar-admin namespaces unsubscribe -s sub tn1/ns1`. 
However, role(not super-user or administrator) with `consume` auth action for 
namespace cannot perform `unsubscribe` operation when enable auth.
   
   The root of the problem is that `PulsarAuthorizationProvider` lacks support 
for namespace operation `UNSUBSCRIBE` when verifying the role's authorization, 
code as below: 
   
https://github.com/apache/pulsar/blob/8cae63557a318240e95697f382b4f61c22b70d64/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/NamespacesBase.java#L1667-L1669
   
https://github.com/apache/pulsar/blob/8cae63557a318240e95697f382b4f61c22b70d64/pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/PulsarAuthorizationProvider.java#L522-L536
   
   The purpose of this PR is to support that role with `consume` namespace 
authorization could `unsubscribe` subscriptions on a namespace.
   
   ### Documentation 
   - [x] `no-need-doc` 
    


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to