Anonymitaet commented on a change in pull request #7251:
URL: https://github.com/apache/pulsar/pull/7251#discussion_r551895241
##########
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:
Hi @rdhabalia
Since this code change affects both REST API and pulsar-admin docs, could
you please update the docs in this PR as below? Many thanks~
- REST API
Go to [TenantsBase.java, Line
69](https://github.com/apache/pulsar/blob/master/pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/TenantsBase.java#L69),
add `This operation requires admin privilege` after `Get the admin
configuration for a given tenant.`
- pulsar-admin
Go to [CmdTenants.java, Line
43](https://github.com/apache/pulsar/blob/master/pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdTenants.java#L43),
add `This operation requires admin privilege` after `Gets the configuration of
a tenant.`
(P.S. I've confirmed this w/ @tuteng)
----------------------------------------------------------------
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:
[email protected]
