Commite6d4e8b

committed

Checking Content-Type in response before assuming json when api call error occurs

1 parent1eaf463 commite6d4e8bCopy full SHA for e6d4e8b

File tree

-1

lines changed

-1

lines changed

Lines changed: 4 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -108,7 +108,10 @@ def make_request(self, fn, endpoint, as_json, **kwargs):`
`108`	`108`
`109`	`109`	`# We should raise a custom exception if status code is not 20X`
`110`	`110`	`ifnot200<=r.status_code<300:`
`111`		`-error_message=r.json().get('message','API error.')`
	`111`	`+ifr.headers.get('Content-Type','').startswith('application/json'):`
	`112`	`+error_message=r.json().get('message','API error.')`
	`113`	`+else:`
	`114`	`+error_message='API error.'`
`112`	`115`	`# Raised exception will be caught and retried`
`113`	`116`	`raiseApiCall.get_exception(r.status_code)(r.status_code,error_message)`
`114`	`117`

Comments

(0)