httpcore
httpcore copied to clipboard
added timeouts to socks5 connection initialization
Summary
added timeouts to socks5 connection initialization
Checklist
- [x] I understand that this PR may be closed in case there was no previous discussion. (This doesn't apply to typos!)
- [x] I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
- [x] I've updated the documentation accordingly.
this fixes issue #1015
Thanks! I faced this issue today. Socks proxy doesn't respond and the code stuck on awaiting stream.read much longer than I expected passing the explicit timeout to the request.
I hope this fix is merges soon. Looking forward.