From c8aec7d40f18af41104b8b86f103e526429d88fc Mon Sep 17 00:00:00 2001 From: Florian Schieder Date: Tue, 11 Mar 2025 06:14:04 +0100 Subject: [PATCH] fixed django middleware: add query to debug log correctly This has likely never worked correctly, as Django's query log entry has ever since been a dictionary made of 'time' and 'sql'. --- .../google/cloud/sqlcommenter/django/middleware.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/python/sqlcommenter-python/google/cloud/sqlcommenter/django/middleware.py b/python/sqlcommenter-python/google/cloud/sqlcommenter/django/middleware.py index 90d80995..bc17c404 100644 --- a/python/sqlcommenter-python/google/cloud/sqlcommenter/django/middleware.py +++ b/python/sqlcommenter-python/google/cloud/sqlcommenter/django/middleware.py @@ -92,6 +92,6 @@ def __call__(self, execute, sql, params, many, context): # Add the query to the query log if debugging. if isinstance(context['cursor'], CursorDebugWrapper): - context['connection'].queries_log.append(sql) + context['connection'].queries_log.append({"sql": sql}) return execute(sql, params, many, context)