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 2021/09/07 10:10:43 UTC

[GitHub] [pulsar] BewareMyPower commented on issue #11945: Admin/Permissions: getPartitionedTopicListshould work even with Topic Level Permissions

BewareMyPower commented on issue #11945:
URL: https://github.com/apache/pulsar/issues/11945#issuecomment-914176801


   I'm not so familiar with the authentication part. I only know the admin permission cannot only be granted to the tenant role. And `AuthAction` doesn't have admin related permissions.
   
   https://github.com/apache/pulsar/blob/981cb626e59267e38c3b29faa02c14f0848a530f/pulsar-client-admin-api/src/main/java/org/apache/pulsar/common/policies/data/AuthAction.java#L24-L42
   
   IMO we can add a readonly admin permission for non-admin roles, like lookup for topics. But it looks like to be a significant change for Pulsar. It might need a discussion for opinions from others.


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