Skip to content
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

Feature Request: add socks5 support for connecting via socks5 proxy #3241

Open
dmitryroshchin opened this issue May 21, 2024 · 0 comments
Open

Comments

@dmitryroshchin
Copy link

Version: 5.0.4

Platform: MacOS Monterey

Description: I've bumped into a problem while trying to use iredis which depends on redis-py to interact with elasticache cluster via ssh tunnel when only a single port on localhost can be used.
Obvious solution was to use socks5 proxy behind the tunnel. The main obstacle is inability of the redis-py to connect via socks5 proxy.

Seems that adding it requires just a few lines of code

The patch above adds support for using sock5 proxy if variables REDIS_SOCKS5_PROXY_HOST and REDIS_SOCKS5_PROXY_PORT are set. Using of variables makes it work with minimal code changes.

The patch was tested on MacOS Monterey
I believe the feature would be very useful for lots of people

Is there any chance of these changed to be accepted via PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant