Requests management - Githubissues

(This issue should be created in the pyrestcli repo, but we are not managing it right now)

We are managing responses in different ways, catching some exception and passing others:

We have 3 different approaches (at least) to manage responses, it could be reasonable but at least, there are problems and bugs managing errors, because the idea is to manage errors using:

pyrestcli with get_response_data https://github.com/danicarrion/pyrestcli/blob/master/pyrestcli/auth.py#L38-L62

but sometimes we are not using it (the client should do it explicitly if you are using the Manager for example) and in some cases, we are managing other cases here like CartoRateLimitException https://github.com/CartoDB/carto-python/blob/master/carto/auth.py#L151-L152

I don't want to write the solution here, at least I am going to say we should have a unique place where:

to check if a response is valid or not
manage the exceptions (like the get_response_data is doing right now)
doing the check by default in every request

CartoDB / carto-python

Requests management #122