wolfstudy commented on PR #25575: URL: https://github.com/apache/pulsar/pull/25575#issuecomment-4311372933
> In addition, it would be useful to create a separate PR to add Socks proxy support for https serviceUrls. The Socks proxy in the Pulsar client is only used for the Pulsar binary protocol at the moment. In that case, there would be a need to specify whether the socks proxy should be used for both http lookups as well as Pulsar binary protocol connections to brokers. The Pulsar client has also http clients in failover implementations. Thanks lhotari. That is a great suggestion. Adding a toggle switch here to control different behaviors is indeed a better implementation approach. I will readjust and revise accordingly. -- 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]
