Do round trip with json dumps and loads to ensure true json params

pycontribs · Dec 21, 2023 · ba1e821 · ba1e821
1 parent a01fc15
commit ba1e821
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/jira/client.py b/jira/client.py
@@ -717,7 +717,13 @@ def _fetch_pages(
             except ImportError:
                 pass
             async_workers = self._options.get("async_workers")
-        page_params = params.copy() if params else {}
+
+        def json_params() -> dict[str, Any]:
+            # passing through json.dumps and json.loads ensures json
+            return json.loads(json.dumps(params.copy())) if params else {}
+
+        page_params = json_params()
+
         if startAt:
             page_params["startAt"] = startAt
         if maxResults:
@@ -766,7 +772,7 @@ def _fetch_pages(
                         session=self._session, max_workers=async_workers
                     )
                     for start_index in range(page_start, total, page_size):
-                        page_params = params.copy() if params else {}
+                        page_params = json_params()
                         page_params["startAt"] = start_index
                         page_params["maxResults"] = page_size
                         url = self._get_url(request_path)
@@ -787,7 +793,7 @@ def _fetch_pages(
                     and len(next_items_page) == page_size
                 ):
                     page_params = (
-                        params.copy() if params else {}
+                        json_params()
                     )  # Hack necessary for mock-calls to not change
                     page_params["startAt"] = page_start
                     page_params["maxResults"] = page_size