nodece opened a new pull request, #16645:
URL: https://github.com/apache/pulsar/pull/16645

   Fixes #16574 
   
   ### Motivation
   
   The `MultiRolesTokenAuthorizationProvider` cannot handle the superuser and 
tenant admin correctly. It only checks if the role passed in is a super user, 
which is incorrect. The correct way is to get a role list from authentication 
data and then check if the role list contains a super user.
   
   ### Modifications
   
   - Override the `isSuperAdmin`
   - Override the `validateTenantAdminAccess`
   - Add test for the `MultiRolesTokenAuthorizationProvider`
   
   ### Documentation
   
   - [x] `doc-not-needed` 
   


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