-
Notifications
You must be signed in to change notification settings - Fork 221
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
Handle ssl connections #284
Comments
from my point of view, I never saw the need to connect Redis through SSL, since it typically runs on the same host as the application server. How is your use-case / architecture? |
In our usecase, we use a managed Redis on Azure platform. By default Azure, only opens the 6380 port (dedicated to SSL connections. We don't want to use non-ssl port (6379) as we don't have a private network in this usecase. |
OK. This is a really valid use-case. I would have to dig into this topic. Do you have time to create a pull-request? |
Yes i'll do it today |
* `RedisPublisher` handles `ssl` flag * Update docs
* `RedisPublisher` handles `ssl` flag * Update docs
As far as i understand, it looks impossible to use this library with SSL redis connections.
The reason is that
settings.WS4REDIS_CONNECTION
is used to instantiate:The reason is that those two objects does not have the same signature for handling ssl connections:
connection_class
argumentRedis
has a simplessl
flagI propose to use one of the two conventions (the
ssl
flag for instance which is easier to understand) and adapt code inwsgi_server.py
.What do you think?
The text was updated successfully, but these errors were encountered: