From bb82f10cd649a690c3c3b0e5eb1c55d9271bd556 Mon Sep 17 00:00:00 2001 From: Egor fureunoir Gorbunov Date: Sun, 6 Jul 2025 17:18:53 +0300 Subject: [PATCH] Features: 1) Add `mail_admins` logging handler for error notifications; 2) Introduce `django.server` logger configuration with INFO level; Fixes: 1) Streamline logger handlers by replacing specific debug/production configuration with unified `console` handler; 2) Update template logger level to ERROR in production settings; Extra: Removed unused `console_debug` and `console_production` handlers for cleaner configuration; --- evibes/settings/logconfig.py | 50 +++++++++++++++++------------------- 1 file changed, 23 insertions(+), 27 deletions(-) diff --git a/evibes/settings/logconfig.py b/evibes/settings/logconfig.py index bdce7eb8..e52a681a 100644 --- a/evibes/settings/logconfig.py +++ b/evibes/settings/logconfig.py @@ -40,87 +40,83 @@ LOGGING = { }, }, "filters": { - "require_debug_true": { - "()": "django.utils.log.RequireDebugTrue", - }, "skip_variable_doesnotexist": { "()": "evibes.settings.logconfig.SkipVariableDoesNotExistFilter", }, }, "handlers": { - "console_debug": { - "level": "DEBUG", - "filters": ["require_debug_true"], + "console": { "class": "logging.StreamHandler", "formatter": "color", }, - "console_production": { - "level": "WARNING", - "class": "logging.StreamHandler", - "formatter": "color", + "mail_admins": { + "level": "ERROR", + "filters": ["require_debug_false"], + "class": "django.utils.log.AdminEmailHandler", }, }, "loggers": { "django": { "handlers": [ - "console_debug" if DEBUG else "console_production", + "console", + "mail_admins", ], - "level": "DEBUG" if DEBUG else "INFO", # noqa: F405 + "level": "DEBUG" if DEBUG else "INFO", "propagate": True, }, + "django.server": { + "level": "INFO", + "propagate": False, + }, "django.request": { "handlers": [ - "console_debug" if DEBUG else "console_production", + "console", + "mail_admins", ], "level": "DEBUG" if DEBUG else "INFO", "propagate": False, }, "django.db.backends": { "handlers": [ - "console_debug" if DEBUG else "console_production", + "console", + "mail_admins", ], "level": "WARNING", "propagate": False, }, "django.template": { "handlers": [ - "console_debug" if DEBUG else "console_production", + "console", + "mail_admins", ], - "level": "DEBUG" if DEBUG else "INFO", + "level": "DEBUG" if DEBUG else "ERROR", "propagate": True, "filters": ["skip_variable_doesnotexist"], }, - "evibes": { - "handlers": [ - "console_debug" if DEBUG else "console_production", - ], - "level": "DEBUG" if DEBUG else "WARNING", - "propagate": True, - }, "django_elasticsearch_dsl": { "handlers": [ - "console_debug" if DEBUG else "console_production", + "console", ], "level": "WARNING", "propagate": False, }, "celery.app.trace": { "handlers": [ - "console_debug" if DEBUG else "console_production", + "console", ], "level": "DEBUG" if DEBUG else "INFO", "propagate": False, }, "celery.worker.strategy": { "handlers": [ - "console_debug" if DEBUG else "console_production", + "console", ], "level": "DEBUG" if DEBUG else "INFO", "propagate": False, }, "elastic_transport.transport": { "handlers": [ - "console_debug" if DEBUG else "console_production", + "console", ], "level": "ERROR", "propagate": False,