With the current implementation, any error response with a non-empty body will be considered a success unless it contains the string "Service Unavailable". This includes, for example, a 502 Bad Gateway response with an HTML body, causing other code to consider the call a success and fail when trying to treat the HTML response as a dict.
This change makes sure HTTP errors are never considered successful requests.
With the current implementation, any error response with a non-empty body will be considered a success unless it contains the string "Service Unavailable". This includes, for example, a 502 Bad Gateway response with an HTML body, causing other code to consider the call a success and fail when trying to treat the HTML response as a dict.
This change makes sure HTTP errors are never considered successful requests.