Skip to content

Add ASGI-native logging pipeline#130

Merged
vishalanandl177 merged 1 commit into
mainfrom
SCRUM-13-P1-Add-ASGI-native-logging-pipeline
Jul 4, 2026
Merged

Add ASGI-native logging pipeline#130
vishalanandl177 merged 1 commit into
mainfrom
SCRUM-13-P1-Add-ASGI-native-logging-pipeline

Conversation

@vishalanandl177

@vishalanandl177 vishalanandl177 commented Jul 4, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add a dual sync/async APILoggerMiddleware entrypoint for ASGI-served Django/DRF requests while preserving existing sync middleware behavior.
  • Share resolver, tracing/correlation, profiling, log construction, database enqueue, and signal emission logic between sync and async paths.
  • Add ASGI coverage for masking, queue failure isolation, content-type handling, truncation, path formatting, custom tracing, queue thresholds, and concurrent context isolation.
  • Add ASGI setup docs and a scripts/measure_asgi_overhead.py benchmark helper.

Jira: https://coderssecret.atlassian.net/browse/SCRUM-13

Validation

  • coverage run --source=drf_api_logger -m django test tests --settings=tests.test_settings --verbosity=1; coverage report -> 270 tests OK, total package coverage 93%, drf_api_logger\middleware\api_logger_middleware.py 100%.
  • python -m build --sdist --wheel; python -m twine check dist\* -> wheel and sdist passed.
  • git diff --cached --check -> passed before commit.
  • PYTHONPATH=J:\projects\DRF-API-Logger J:\projects\drf-demo\venv\Scripts\python.exe manage.py test books --verbosity=1 -> 3 tests OK.

Local Matrix Exception

  • tox --version -> tox command not found in this shell.
  • python -m tox --version -> No module named tox.

@vishalanandl177
vishalanandl177 merged commit 6b66764 into main Jul 4, 2026
6 checks passed
@vishalanandl177
vishalanandl177 deleted the SCRUM-13-P1-Add-ASGI-native-logging-pipeline branch July 4, 2026 14:04
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.

1 participant