bugfix: get_results_csv() use response.content, not response.raw (#52)

msf · web-flow · commit c98dd4e32557 · 2023-03-20T21:52:50.000+01:00
diff --git a/dune_client/client.py b/dune_client/client.py
@@ -120,7 +120,7 @@ def get_result_csv(self, job_id: str) -> ExecutionResultCSV:
             timeout=self.DEFAULT_TIMEOUT,
         )
         response.raise_for_status()
-        return ExecutionResultCSV(data=BytesIO(response.raw))
+        return ExecutionResultCSV(data=BytesIO(response.content))
 
     def cancel_execution(self, job_id: str) -> bool:
         """POST Execution Cancellation to Dune API for `job_id` (aka `execution_id`)"""

Original file line number	Diff line number	Diff line change
`@@ -120,7 +120,7 @@ def get_result_csv(self, job_id: str) -> ExecutionResultCSV:`
`120`	`120`	`timeout=self.DEFAULT_TIMEOUT,`
`121`	`121`	`)`
`122`	`122`	`response.raise_for_status()`
`123`		`- return ExecutionResultCSV(data=BytesIO(response.raw))`
	`123`	`+ return ExecutionResultCSV(data=BytesIO(response.content))`
`124`	`124`
`125`	`125`	`def cancel_execution(self, job_id: str) -> bool:`
`126`	`126`	"""POST Execution Cancellation to Dune API for `job_id` (aka `execution_id`)"""