Skip to content

Commit 2668d64

Browse files
committed
use new sync API endpoints for job/task wait()
1 parent c7dfe1d commit 2668d64

5 files changed

Lines changed: 22 additions & 5 deletions

File tree

cloudconvert/cloudconvertrestclient.py

Lines changed: 14 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111

1212
import cloudconvert.utils as util
1313
from cloudconvert.exceptions import exceptions
14-
from cloudconvert.config import __version__, __endpoint_map__
14+
from cloudconvert.config import __version__, __endpoint_map__, __sync_endpoint_map__
1515

1616
log = logging.getLogger(__name__)
1717

@@ -40,6 +40,7 @@ def __init__(self, options=None, **kwargs):
4040
"Required: live or sandbox")
4141

4242
self.endpoint = kwargs.get("endpoint", self.default_endpoint())
43+
self.sync_endpoint = kwargs.get("sync_endpoint", self.default_sync_endpoint())
4344
# Mandatory parameter, so not using `dict.get`
4445
self.proxies = kwargs.get("proxies", None)
4546
self.token_hash = None
@@ -57,6 +58,9 @@ def __init__(self, options=None, **kwargs):
5758
def default_endpoint(self):
5859
return __endpoint_map__.get(self.mode)
5960

61+
def default_sync_endpoint(self):
62+
return __sync_endpoint_map__.get(self.mode)
63+
6064
def request(self, url, method, body=None, headers=None):
6165
"""Make HTTP call, formats response and does error handling. Uses http_call method in CloudConvertRestClient class.
6266
Usage::
@@ -161,6 +165,15 @@ def get(self, action, headers=None):
161165
"""
162166
return self.request(util.join_url(self.endpoint, action), 'GET', headers=headers or {})
163167

168+
169+
def get_sync(self, action, headers=None):
170+
"""Make GET request to sync API
171+
Usage::
172+
>>> cloudconvertrestclient.get_sync("v2/tasks/TASK-ID")
173+
>>> cloudconvertrestclient.get_sync("v2/jobs/JOB-ID")
174+
"""
175+
return self.request(util.join_url(self.sync_endpoint, action), 'GET', headers=headers or {})
176+
164177
def post(self, action, params=None, headers={}):
165178
"""Make POST request
166179
Usage::

cloudconvert/config.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,4 +7,8 @@
77
"live": "https://api.cloudconvert.com",
88
"sandbox": "https://api.sandbox.cloudconvert.com"
99
}
10+
__sync_endpoint_map__ = {
11+
"live": "https://sync.api.cloudconvert.com",
12+
"sandbox": "https://sync.api.sandbox.cloudconvert.com"
13+
}
1014
SANDBOX_API_KEY = "eyJ0eXAiOiJKV1QiLCJhbGciOiJSUzI1NiIsImp0aSI6IjI4YmE3OGQyZjc1NWM5ZGE3Yjg1NDRhMWRkMjg2NWM4N2U0YzI5NWI0NzQ0Zjc4ZDNmMzA3OWM2NjU3ZjI0MjVhOTMyYjIxMjU5ZGU2NWQ4In0.eyJhdWQiOiIxIiwianRpIjoiMjhiYTc4ZDJmNzU1YzlkYTdiODU0NGExZGQyODY1Yzg3ZTRjMjk1YjQ3NDRmNzhkM2YzMDc5YzY2NTdmMjQyNWE5MzJiMjEyNTlkZTY1ZDgiLCJpYXQiOjE1NTkwNjc3NzcsIm5iZiI6MTU1OTA2Nzc3NywiZXhwIjo0NzE0NzQxMzc3LCJzdWIiOiIzNzExNjc4NCIsInNjb3BlcyI6WyJ1c2VyLnJlYWQiLCJ1c2VyLndyaXRlIiwidGFzay5yZWFkIiwidGFzay53cml0ZSIsIndlYmhvb2sucmVhZCIsIndlYmhvb2sud3JpdGUiXX0.IkmkfDVGwouCH-ICFAShQMHyFAHK3y90CSoissUVD8h5HFG4GqN5DEw0IFzlPr1auUKp3H1pAvPutdIQtrDMTmUUmGMUb2dRlCAuQdqxa81Q5KAmcKDgOg2YTWOWEGMy3jETTb7W6vyNGsT_3DFMapMdeOw1jdIUTMZqW3QbSCeGXj3PMRnhI7YynaDtmktjzO9IUDHbeT2HRzzMiep97KvVZNjYtZvgM-kbUjE6Mm68_kA8JMuQeor0Yg7896JPV0YM3-MnHf7elKgoCJbfBCDAbvSX_ZYsSI7IGoLLb0mgJVfFcH_HMYAHhJj5cUEJN2Iml-FkODqrRk72bVxyJs9j1GPQBl4ORXuU9yrjUgHrRaZ5YM__LwsUQB3AuB92oyQseCjULn1sWM1PzIXCcyVjKZSpn9LAAGNf9paCF-_G9ok9tZKccRouCiYl9v5XbmuxV8hXYp6fXZxyaAkj_JN2kErVSkxYzVyyZL1e220aFFnbch6nDvLFHgi-WeTQHFQDzuHsM8RKRixV8uD7pk3de4AEYg0EWqZHCr82qY7TGdSQvuAS0QIy3B89OwQW0ROW4k3Yw0XIKgKSYWyKnc7huc7yPQUIDDDAOa5OojXrVY5ZuL_hwQMIOmejcHTKFdAgzAaVnRkC8_FfVh4wHCPBaHjze9hRp5n4O1pnPFI"

cloudconvert/resource.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -184,8 +184,8 @@ def wait(cls, id):
184184
"""
185185
api_client = default_client()
186186

187-
url = util.join_url(cls.path, str(id), "wait")
188-
res = api_client.get(url)
187+
url = util.join_url(cls.path, str(id))
188+
res = api_client.get_sync(url)
189189
try:
190190
return res["data"]
191191
except:

tests/unit/testJob.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ def testWaitJob(self):
7070
response_json = json.load(f)
7171

7272
job_id = "4c80f1ae-5b3a-43d5-bb58-1a5c4eb4e46b"
73-
m.get("https://api.sandbox.cloudconvert.com/v2/jobs/{}/wait".format(job_id), json=response_json)
73+
m.get("https://sync.api.sandbox.cloudconvert.com/v2/jobs/{}".format(job_id), json=response_json)
7474

7575
job = self.cloudconvert.Job.wait(id=job_id)
7676

tests/unit/testTask.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ def testWaitTask(self):
6464
response_json = json.load(f)
6565

6666
task_id = "4c80f1ae-5b3a-43d5-bb58-1a5c4eb4e46b"
67-
m.get("https://api.sandbox.cloudconvert.com/v2/tasks/{}/wait".format(task_id), json=response_json)
67+
m.get("https://sync.api.sandbox.cloudconvert.com/v2/tasks/{}".format(task_id), json=response_json)
6868

6969
task = self.cloudconvert.Task.wait(id=task_id)
7070

0 commit comments

Comments
 (0)