safe-global / safe-decoder-service / 24830722284

92%

Build Type

push

github

Committed by Uxio0

Commit Message

Merge log_record_factory so both db_session and task_detail are injected

main.py and workers/tasks.py each registered their own log record factory
via logging.setLogRecordFactory, with the last import winning. In the web
process (which transitively imports workers/tasks.py via events.py) the
task factory was overwriting the request factory, causing db_session to be
absent from web logs.

A single log_record_factory is now defined in safe_logger.py and registered
there. It lazily imports _db_session_context from database.py (avoiding the
config -> safe_logger -> database circular import) and reads both context
vars, so all log records carry whichever attributes are active in the current
context.

Coverage Stats

12 of 12 new or added lines in 2 files covered. (100.0%)

1 existing line in 1 file now uncovered.

1309 of 1432 relevant lines covered (91.41%)

0.91 hits per line