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 2022/12/15 06:44:48 UTC

[GitHub] [pulsar] tuteng commented on pull request #18794: [fix][client] Fix update response body text

tuteng commented on PR #18794:
URL: https://github.com/apache/pulsar/pull/18794#issuecomment-1352627209

   > > When using pulsar-admin and encountering an exception, first check the body, then check the http text
   > 
   > Can you give an actual case this PR is solving?
   
   https://github.com/apache/pulsar/blob/master/pulsar-broker/src/main/java/org/apache/pulsar/broker/web/ExceptionHandler.java#L46 
   
   This needs an interceptor and then throws an exception with a custom message, using pulsar-admin to execute the corresponding command, which never gets the custom message @Jason918 


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