epicserve / django-cache-url

Use Cache URLs in your Django Application
MIT License
18 stars 12 forks source link

Add support for Redis over SSL #5

Closed astei closed 3 years ago

astei commented 4 years ago

This is required for certain cloud setups (DigitalOcean managed Redis, AWS ElastiCache).

astei commented 4 years ago

I have tested rediss. hirediss hasn't been tested but is suspected to work.

epicserve commented 4 years ago

@astei, I'm reluctant to merge this in because I would rather have issue #1 and PR #4 merged in first. Want to help review and comment on those or even add documentation?

incorvia commented 3 years ago

For now I'm just targeting this PR in my project. I'm using Saleor which relies on this project and we can't host anywhere using 12 factor without this getting merged. What can we do to get this merged?

syphar commented 3 years ago

We're also just stumbling on this issue, using Heroku. @epicserve what could we do to get this merged / released?

We would be happy to invest time into resolving this on the library level, while we don't need #1 or #4 :)

epicserve commented 3 years ago

I just released v3.2.0 which adds support for django-redis-cache and SSL. I already had a PR that added both features so I decided to update it and finish it. Thank you for your contribution, if you find any other bugs PRs are welcome.