-
-
Notifications
You must be signed in to change notification settings - Fork 9.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes issue #3863 #4229
Fixes issue #3863 #4229
Conversation
How tests are failing for Python 3.3. I have only changed one .rst on the docs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think the test failures will be relevant.
docs/user/advanced.rst
Outdated
@@ -593,6 +593,10 @@ as using a HTTP one:: | |||
'https': 'socks5://user:pass@host:port' | |||
} | |||
|
|||
Using socks5 proxy resolve domains through the client instead of the proxy server. This is done to be | |||
in line with curl, which uses different schemes to decide whether to resolve via the proxy or via the client. | |||
You need to use the scheme socks5h is you want to resolve domains through the proxy server. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's change this to:
Using the scheme
socks5
causes the DNS resolution to happen on the client, rather than on the proxy server. This is in line with curl, which uses the scheme to decide whether to do the DNS resolution on the client or proxy. If you want to resolve the domains on the proxy server, usesocks5h
as the scheme.
I update the docs accordingly. |
docs/user/advanced.rst
Outdated
@@ -593,6 +593,8 @@ as using a HTTP one:: | |||
'https': 'socks5://user:pass@host:port' | |||
} | |||
|
|||
Using the scheme socks5 causes the DNS resolution to happen on the client, rather than on the proxy server. This is in line with curl, which uses the scheme to decide whether to do the DNS resolution on the client or proxy. If you want to resolve the domains on the proxy server, use socks5h as the scheme. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we surround socks
and socks5
with double-backticks? That is, make it:
``socks``
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you mean socks5
and socks5h
right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I do. ;)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perfect, thankyou so much!
Update the docs for issue #3863 .