Skip to content

Commit d7e42e1

Browse files
committed
Updates based on feed back from @drew-russell
1 parent 07339e1 commit d7e42e1

1 file changed

Lines changed: 3 additions & 4 deletions

File tree

rubrik_cdm/api.py

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -184,20 +184,19 @@ def _common_api(self, call_type, api_version, api_endpoint, config=None, job_sta
184184
if call_type == "QUERY":
185185
try:
186186
error_message
187-
raise BaseException
187+
raise APICallException(error_message)
188188
except NameError:
189189
try:
190190
return api_request.json()["data"]
191191
except BaseException:
192192
pass
193193

194-
# request.json() will fail on a 204 (No Content), so return an empty string
194+
# request.json() will fail on a 204 (No Content), so just the response code
195195
if api_request.status_code != 204:
196196
return api_request.json()
197197
else:
198-
return ""
198+
return {'status_code': api_request.status_code}
199199
except BaseException:
200-
raise APICallException(error_message)
201200
return {'status_code': api_request.status_code}
202201

203202
def get(self, api_version, api_endpoint, timeout=15, authentication=True, params=None):

0 commit comments

Comments
 (0)