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/08/29 16:24:23 UTC

[GitHub] [pulsar] frankjkelly commented on a change in pull request #7251: [pulsar-admin] Fix tenant admin should be able to get tenant-info

frankjkelly commented on a change in pull request #7251:
URL: https://github.com/apache/pulsar/pull/7251#discussion_r479665515



##########
File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/TenantsBase.java
##########
@@ -76,7 +76,7 @@
     public TenantInfo getTenantAdmin(
         @ApiParam(value = "The tenant name")
         @PathParam("tenant") String tenant) {
-        validateSuperUserAccess();
+        validateAdminAccessForTenant(tenant);

Review comment:
       This change is pretty smart - is there any documentation that needs to change as a result (from personal experience setting up AuthN/AuthZ is pretty hard outside of a standalone case). Thanks!




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