Replies: 2 comments
-
could you proceed with the suggested improvements as a pr? |
Beta Was this translation helpful? Give feedback.
0 replies
-
moving this to discussion for now |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Common for django projects is:
celery_app.config_from_object('django.conf:settings', namespace='CELERY')
The problem here is IMHO that the complete django settings will be consume and not only the settings starts with
CELERY_
e.g.: The output of celery inspect report it totally long and confusing.
And if may be a security issues that something like SECRET_KEY is consumed. (However, the entry in the report is replaced by
'********'
.What's about to filter by namespace in the loader?
Beta Was this translation helpful? Give feedback.
All reactions