-
-
Notifications
You must be signed in to change notification settings - Fork 521
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
Add support for HTTP(S) proxies to connect() #364
Comments
Are you making connections with |
yep. for example, i want there exists function like because my running environment is in the mainland of china where limits connections to lots of hosts overseas. I have to make websocket connections through my socks5 proxy which communicates with shadowsocksR server. |
OK. This isn't currently implemented. |
This feature request is for implemting: https://tools.ietf.org/html/rfc7231#section-4.3.6 We must take care of handling TLS properly:
This may require handling the TLS handshake manually where |
I have a patch that manages the four cases: TLS / not TLS between client and proxy / proxy and server. The patch needs tests and I'm not sure how to write them. |
TLS between proxy and server is very messy due to https://bugs.python.org/issue23749 Also we're likely to run into issues like aio-libs/aiosmtpd#83 until we add code to handle that. |
According to https://bugs.python.org/issue23749 the bug was fixed as of 2018-05-28. Might be worth retaking a look at? @aaugustin |
I can consider that bug resolved after I drop support for all Python versions before 3.7, about five years from now. |
Any news? |
No. |
Would supporting only client proxies to which you connect over HTTP be any easier? From the discussion above, it looks like the difficulties were all when connecting to the proxy over HTTPS. I ask because, at least as I understand it, an proxy to which you connect over HTTP is still pretty secure, even when you're using it to ultimately connect to a site over HTTPS. The pattern is: Client makes an HTTP connection to the proxy and sends a message like
The proxy, if it's happy to proxy the request, and is able to connect to that host on that port, replies
...and then the proxy just forwards data blindly down the pipe. The data that is sent and received can be TLS (or indeed SSH or any other TCP-based protocol). It's worth noting that everything is reasonably secure. The only information that is sent in the clear from the client to the proxy is the port number (which is effectively in the clear for all TCP connections anyway) and the hostname, which TLS handshakes already send in the clear via the SNI field. It's possible (or perhaps likely) that I'm missing something important here, though. |
Partial support would be better than no support at all, indeed. However, since there's a working PoC in #422 that does what you said and is merely missing error handling and tests, once it's complete for HTTP proxies, I don't think it would be very hard to also complete it for HTTP proxies. |
For anyone still looking for this, I recommend using the |
@nyuszika7h Proxies are a pain. If you find this laughable easy, then just make a PR. PRs are always welcome. The only thing distasteful here is your comment. |
I didn't say HTTPS proxies are trivial, but definitely not the kind of thing you need to beg for funding for. There's plenty of prior art to look at. |
"The community house in this area could use some cleaning. Are you available tomorrow? Don't ask for payment though, cleaning has already been done to other community houses in the past." |
@nyuszika7h This is basically what the last comment before yours said — glad to see that we agree on this1 |
watching this, any update? |
Hello from 2023! I see this comment was made in 2018 :) |
Good job following up :-) No specific plans in this area at the time being, though. |
@kyochikuto @weaming hey. You can check out my package: https://github.com/racinette/websockets_proxy . I subclassed the original |
Nic job! Although I'm not writing Python code now. You guys can try this. |
Five years have passed. |
Yes, that obstacle no longer exists. |
I just tested this and it's working flawlessly, really appreciate it! I love how it is only a wrapper around the original module and not a modified fork, this way you can easily keep it updated with the original module. Keep up the good job! |
@kyochikuto thanks man, appreciate it. It can be used as drop-in replacement via monkey-patching, in case some other package using Also, please, do star the repo, so I can gain recognition. |
Drop-in replacements are always preferred, thanks!
Done! |
dude, it's been 6 years. |
dude, it's just a hobby, I have a real job :-) |
how to make the websocket connection go through with http proxy?
The text was updated successfully, but these errors were encountered: