poorbarcode commented on PR #19830: URL: https://github.com/apache/pulsar/pull/19830#issuecomment-1492262201
Hi @michaeljmarshall I saw that you [pushed a commit into `branch-2.10`](https://github.com/apache/pulsar/commit/7ea5f43feeef7d91d89cd187cecbd764c84db396) instead of cherry-picking #19830. But there was a mistake that caused `branch-master` and `branch-2.10` to be implemented differently, see: https://github.com/apache/pulsar/commit/7ea5f43feeef7d91d89cd187cecbd764c84db396#diff-73e921d897fa80c701149187144cdd4141c719040775e3b8258c5a8823a08bdcR198, and it makes an issue: Non-super users can not operate the tenant API even if `authorization` is disabled (but `authentication` is enabled). You can reproduce like this: enabled authentication and disabled authorization; call `pulsar-admin tenants list` with the non-super role. -- 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]
