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/28 22:03:26 UTC

[GitHub] [pulsar] michaeljmarshall commented on pull request #11872: [Broker] Update Authz Check Before Completing Topic Level Policy Operation

michaeljmarshall commented on pull request #11872:
URL: https://github.com/apache/pulsar/pull/11872#issuecomment-929658562


   @315157973 and @KannarFr - what is the right path forward here? This PR is at least helpful in adding tests that assert failure when a call is made with insufficient authorization. Do we need some interface implemented or should I close this PR and open a new one with the just the tests? The current implementation actually throws a 401, not a 403 as the annotations indicate, so those might need updating too. 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.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org