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

Change cache_url to redis_url #237

Merged
merged 1 commit into from
Apr 3, 2023
Merged

Change cache_url to redis_url #237

merged 1 commit into from
Apr 3, 2023

Conversation

filippo-20tab
Copy link
Contributor

@filippo-20tab filippo-20tab commented Mar 1, 2023

The current secret holds the CACHE_URL value, as expected by the django-cache-url package. This prevents other potential components (e.g. Celery) to access the Redis server URL, which is contained in the CACHE_URL value but can not easily be used as-is. Therefore I suggest storing a REDIS_URL value with the Redis server URL, and then populate the CACHE_URL environment variable from it in the Django deployment, as proposed in the twin PR in the Django-Talos subrepo.

@trottomv trottomv marked this pull request as ready for review March 31, 2023 12:43
@trottomv trottomv merged commit f6aa429 into main Apr 3, 2023
@trottomv trottomv deleted the feature/redis-url-secret branch April 3, 2023 10:03
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

Successfully merging this pull request may close these issues.

3 participants