Github user ok2c commented on the issue:
https://github.com/apache/httpcomponents-core/pull/94
@dmap My first impression is very positive. Changes look concise and
non-intrusive. I have one conceptual question though. Was there any particular
reason for putting `SOCKS` protocol logic into `InternalChannel` instead of a
custom `IOEventHandler` (similarly to `ClientHttpProtocolNegotiator`)?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]