kobotoolbox / kpi / 22917474912

82%

master: 76%

push

github

fix(logging): prevent duplicate logs by disabling propagation to root logger DEV-1838 (#6808)

### 📣 Summary

Celery workers were emitting every log record twice, polluting logs and
making debugging harder.

### 📖 Description

When a logger is configured with `propagate: True` (the default), log
records are passed up the logger hierarchy all the way to the root
logger. Because Django's logging setup attaches a handler both on the
named logger and on the root logger, each record was processed twice —
once by the named handler and once by the root handler — resulting in
every log line appearing duplicated.

Setting `propagate: False` on `console_logger` stops records from
bubbling up, so each log record is handled exactly once.

### 💭 Notes

- One-line change in `LOGGING['loggers']['console_logger']`: `propagate`
set from `True` to `False`.
- No impact on log levels or log destinations — only the duplication is
removed.

7409 of 11506 branches covered (64.39%)

28495 of 34831 relevant lines covered (81.81%)

5.81 hits per line