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

Incompatibility when using hiredis:// with Django 4 #20

Open
matthiask opened this issue Mar 19, 2022 · 3 comments
Open

Incompatibility when using hiredis:// with Django 4 #20

matthiask opened this issue Mar 19, 2022 · 3 comments

Comments

@matthiask
Copy link

django-cache-url adds PARSER_CLASS to the options dictionary when using hiredis://... but Django's built-in cache backend doesn't like that.

Thanks!

@matthiask
Copy link
Author

By the way, recent enough versions of redis-py will automatically select the hiredis parser if it is available so you should be able to just skip specifying the parser class.

@epicserve
Copy link
Owner

@matthiask, want to make a PR?

@matthiask
Copy link
Author

Thanks for asking! I'll politely decline since I'd rather spend more time on Jazzband projects, e.g. the debug toolbar. django-cache-url has served me well in the past (and still does) but since I've mostly switched to using https://github.com/matthiask/speckenv/blob/main/speckenv_django.py this isn't my own itch anymore. speckenv_django is a bit smaller, doesn't modify urllib.uses_netloc and does everything I need re. configuring databases, email sending and caching.

I hope I don't come across as a freeloader here. Thanks again!

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

2 participants