Log object dicts instead of JSON strings (#85)

Author: leplatrem

src/jbi/runner.py

@@ -36,7 +36,7 @@ def execute_action(
         "bug": {
             "id": request.bug.id if request.bug else None,
         },
-        "request": request.json(),
+        "request": request.dict(),
     }
     try:
         logger.debug(
@@ -53,7 +53,7 @@ def execute_action(
             raise IgnoreInvalidRequestError(
                 "bug not accessible or bugzilla down"
             ) from ex
-        log_context["bug"] = bug_obj.json()
+        log_context["bug"] = bug_obj.dict()
 
         try:
             action_name, current_action = bug_obj.lookup_action(actions)

src/jbi/whiteboard_actions/default.py

@@ -48,7 +48,7 @@ def __call__(  # pylint: disable=inconsistent-return-statements
             "Ignore event target %r",
             target,
             extra={
-                "request": payload.json(),
+                "request": payload.dict(),
             },
         )
 
@@ -58,8 +58,8 @@ def comment_create_or_noop(self, payload: BugzillaWebhookRequest):
         linked_issue_key = bug_obj.extract_from_see_also()
 
         log_context = {
-            "request": payload.json(),
-            "bug": bug_obj.json(),
+            "request": payload.dict(),
+            "bug": bug_obj.dict(),
         }
         if not linked_issue_key:
             logger.debug(
@@ -106,8 +106,8 @@ def bug_create_or_update(
             return self.create_and_link_issue(payload, bug_obj)
 
         log_context = {
-            "request": payload.json(),
-            "bug": bug_obj.json(),
+            "request": payload.dict(),
+            "bug": bug_obj.dict(),
         }
         logger.debug(
             "Update fields of Jira issue %s for Bug %s",
@@ -146,8 +146,8 @@ def create_and_link_issue(
     ):  # pylint: disable=too-many-locals
         """create jira issue and establish link between bug and issue; rollback/delete if required"""
         log_context = {
-            "request": payload.json(),
-            "bug": bug_obj.json(),
+            "request": payload.dict(),
+            "bug": bug_obj.dict(),
         }
         logger.debug(
             "Create new Jira issue for Bug %s",