Updating querystring for logging. (#209)

* Updating querystring for logging. Use of 'dict' resulted in hundreds of additional columns to log schema. Wrapping the 'dict' in a 'str' converts the querystring from a RECORD in BQ to a STRING.

* Updating tests

Author: bsieber-mozilla

jbi/log.py

@@ -54,7 +54,7 @@ def format_request_summary_fields(
         "path": request.url.path,
         "method": request.method,
         "lang": request.headers.get("Accept-Language"),
-        "querystring": dict(request.query_params),
+        "querystring": str(dict(request.query_params)),
         "errno": 0,
         "t": int((current_time - request_time) * 1000.0),
         "time": datetime.fromtimestamp(current_time).isoformat(),

tests/unit/test_main.py

@@ -19,7 +19,7 @@ def test_request_summary_is_logged(caplog):
             assert summary.name == "request.summary"
             assert summary.method == "GET"
             assert summary.path == "/"
-            assert summary.querystring == {}
+            assert summary.querystring == "{}"
 
 
 def test_errors_are_reported_to_sentry(