Skip to content
This repository was archived by the owner on May 29, 2026. It is now read-only.

117 django middleware only applies to default database connections ignoring other databases#127

Merged
sjs994 merged 6 commits into
google:masterfrom
Thiyagu55:117-django-middleware-only-applies-to-default-database-connections-ignoring-other-databases
Jun 17, 2022
Merged

117 django middleware only applies to default database connections ignoring other databases#127
sjs994 merged 6 commits into
google:masterfrom
Thiyagu55:117-django-middleware-only-applies-to-default-database-connections-ignoring-other-databases

Conversation

@Thiyagu55

@Thiyagu55 Thiyagu55 commented May 19, 2022

Copy link
Copy Markdown
Contributor

This PR is a fix to issue #117

Implementation Details:

  1. Added query wrapper over all connections via the existing SQLCommenter midleware

Test Cases Added

  1. test_basic_multiple_db_support
  2. get_query_other_db

Comment thread python/sqlcommenter-python/tests/django/tests.py
…ult-database-connections-ignoring-other-databases
@sjs994 sjs994 merged commit 0c309fc into google:master Jun 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants