django / channels_redis

Redis channel layer backend for Django Channels
BSD 3-Clause "New" or "Revised" License
601 stars 197 forks source link

Document Redis SSL configs #370

Closed bbrowning918 closed 1 year ago

bbrowning918 commented 1 year ago

There is ongoing discussion and help in https://github.com/django/channels_redis/issues/235 and https://github.com/django/channels_redis/issues/330 (are there more?) about how to specify extra config for the connections to Redis, particularly SSL related.

The example provided is how we got Heroku to behave with it's self-signed certs on 4.1.0. Hopefully it will be helpful and answer a common question.

bbrowning918 commented 1 year ago

:disappointed: oh

justuswilhelm commented 1 year ago

May I suggest that there could be a typo in the README as it is right now? (not mentioning the missing restructured text code block)

Right now it says something like the following, which is not a valid dict.

   "hosts":[
        "address": "rediss://user@host:port",  // "REDIS_TLS_URL"
        "ssl_cert_reqs": None
    ]

I reverted to using my own helper for now. It's too bad testing deployments on Heroku is so time consuming.

bbrowning918 commented 1 year ago

@justuswilhelm In hindsight I did not do a great job 'copy-pasting' our config from memory. Thank you for noticing that and the rst syntax mistake.

carltongibson commented 1 year ago

I tweaked it, but happy to take any improvements.