This repository has been archived by the owner on Feb 22, 2023. It is now read-only.
Use get_redis_connection
rather than django.core.cache
for thumbnail timeouts
#1073
Labels
💻 aspect: code
Concerns the software code in the repository
🛠 goal: fix
Bug fix
good first issue
New-contributor friendly
help wanted
Open to participation from the community
🟧 priority: high
Stalls work on the project or its dependents
🐍 tech: python
Requires familiarity with Python
Description
Capturing which domains had thumbnail timeouts on the API was added in #982. However, @sarayourfriend found that using
django.core.cache
implicitly uses the default Django cache timeout. For our setup this is currently 5 minutes, which means that the thumbnail timeout entries only exist in Redis for 5 minutes.We would like to switch the thumbnail timeout capturing from using the default
django.core.cache
to useget_redis_connection
instead, so that no default timeout is applied when the associated values are added or updated.Additional context
This is a stop-gap change, and is intended to be supplanted by #1072
The text was updated successfully, but these errors were encountered: