Merge pull request #50 from JaviCerveraIngram/CPS-27-server-error-response-deserialize

vgrebenschikov · web-flow · commit 3c7299b127c2 · 2019-06-04T14:16:28.000+03:00
Fix for error response.
diff --git a/connect/resources/base.py b/connect/resources/base.py
@@ -102,9 +102,8 @@ def _check_and_pack_response(response):
                 )
 
         if not response.ok:
-            data, error = ServerErrorResponse.deserialize(response.text)
-            if data:
-                raise ServerError(data)
+            error = ServerErrorResponse.deserialize(response.text)
+            raise ServerError(error)
 
         return response.text, response.status_code
 

Original file line number	Diff line number	Diff line change
`@@ -102,9 +102,8 @@ def _check_and_pack_response(response):`
`102`	`102`	`)`
`103`	`103`
`104`	`104`	`if not response.ok:`
`105`		`- data, error = ServerErrorResponse.deserialize(response.text)`
`106`		`- if data:`
`107`		`- raise ServerError(data)`
	`105`	`+ error = ServerErrorResponse.deserialize(response.text)`
	`106`	`+ raise ServerError(error)`
`108`	`107`
`109`	`108`	`return response.text, response.status_code`
`110`	`109`