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/20 11:20:41 UTC

[GitHub] [pulsar] lhotari commented on pull request #17831: [fix][proxy] Fix refresh client auth

lhotari commented on PR #17831:
URL: https://github.com/apache/pulsar/pull/17831#issuecomment-1359207195

   I think that this change should be reverted. This can cause corruption of the traffic between the client and the proxy. 
   I provided some explanation in this comment: https://github.com/apache/pulsar/issues/10816#issuecomment-1359198680 . When the proxy is passing bytes back and forth, it is not safe to inject any protocol messages to the connection. That might corrupt the traffic.


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