From 974f4b3bcfb55d4f3cec61db8a197ed6980268b8 Mon Sep 17 00:00:00 2001
From: Brett Thomas <brettpthomas@gmail.com>
Date: Mon, 30 Dec 2024 14:53:56 -0500
Subject: [PATCH] ensure task data is JSON-serializable when tasks are created

---
 surge/tasks.py | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/surge/tasks.py b/surge/tasks.py
index 046d63d..b771517 100644
--- a/surge/tasks.py
+++ b/surge/tasks.py
@@ -1,4 +1,5 @@
 import dateutil.parser
+import json
 
 from surge.errors import SurgeMissingIDError, SurgeTaskDataError
 from surge.api_resource import PROJECTS_ENDPOINT, TASKS_ENDPOINT, APIResource
@@ -92,6 +93,11 @@ def create(cls, project_id: str, api_key: str = None, **params):
         Returns:
             task: new Task object
         '''
+        try:
+            json.dumps(params)
+        except (TypeError, ValueError):
+            raise SurgeTaskDataError("All task data must be JSON-serializable")
+
         endpoint = f"{PROJECTS_ENDPOINT}/{project_id}/{TASKS_ENDPOINT}"
         data = {"fields": params}
         response_json = cls.post(endpoint, data, api_key=api_key)
@@ -110,6 +116,7 @@ def create_many(cls,
             project_id (str): ID of the project to which the tasks are added.
             tasks_data (list): list of dicts that map each task field to its value.
                 e.g. [{"website": "surgehq.ai"}, {"website":"twitch.tv"}]
+                All values must be JSON-serializable.
 
         Returns:
             tasks (list): list of Task objects
@@ -119,6 +126,11 @@ def create_many(cls,
 
         if not all(isinstance(t, dict) for t in tasks_data):
             raise SurgeTaskDataError
+        
+        try:
+            json.dumps(tasks_data)
+        except (TypeError, ValueError):
+            raise SurgeTaskDataError("All task data must be JSON-serializable")
 
         endpoint = f"{PROJECTS_ENDPOINT}/{project_id}/{TASKS_ENDPOINT}/create_tasks"
         data = {"tasks": tasks_data, "launch": launch}