search

exoscale / python-exoscale

Python bindings for the Exoscale APIs
https://exoscale.github.io/python-exoscale/
ISC License
14 stars 7 forks source link

Error handling #62

Closed elkezza closed 1 month ago

elkezza commented 1 month ago

Adding exception handling, by introducing ExoscaleAPIException as base class for all exoscale exceptions,ExoscaleAPIClientException for handling client-side errors 4xx and ExoscaleAPIServerException for server-side errors 5xx.

Tested them locally and works as expected.

brutasse commented 1 month ago

Fixes #61

elkezza commented 1 month ago

@brutasse thank you for your feedback, I will do accordingly.

elkezza commented 1 month ago

👍 LGTM, you can merge after the final suggested change @elkezza. Thanks!

elkezza commented 1 month ago

👍 LGTM, you can merge after the final suggested change @elkezza. Thanks!

ack, thanks!