Handle ConnectTimeout exceptions in _call method #115
Conversation
correios/client.py
Outdated
| @@ -109,6 +110,8 @@ def _call(self, method_name, *args, **kwargs): | |||
| method = getattr(self.sigep, method_name) | |||
| try: | |||
| return method(*args, **kwargs) | |||
| except ConnectTimeout: | |||
| raise ConnectTimeoutError | |||
| except Fault as exc: | |||
| if 'autenticacao' in str(exc): | |||
There was a problem hiding this comment.
esse é o único/melhor jeito de ver se é um erro de autenticação? não tem nenhum atributo onde eu consiga pegar um código ou a mensagem dentro do objeto exc?
There was a problem hiding this comment.
Yes, executing some tests with SoapClient, to check authentications problems, I just get a Fault exception like this:
Fault: A autenticacao de sigep falhou!
There was a problem hiding this comment.
a parte massa é que quando acontece erro de autenticação a API dos correios retorna código 500 ao invés de 40x 🔝 💯 🎉
correios/client.py
Outdated
| @@ -109,6 +110,8 @@ def _call(self, method_name, *args, **kwargs): | |||
| method = getattr(self.sigep, method_name) | |||
| try: | |||
| return method(*args, **kwargs) | |||
| except ConnectTimeout: | |||
| raise ConnectTimeoutError | |||
There was a problem hiding this comment.
raise ConnectTimeoutError(f"Timeout connection error ({self.timeout} seconds)")Sempre levantamos exceptions como instâncias e não classes. É o momento de informar o usuário detalhes do problema...
|
Done @osantana |
| except Fault as exc: | ||
| if 'autenticacao' in str(exc): | ||
| raise AuthenticationError | ||
| if "autenticacao" in str(exc): |
There was a problem hiding this comment.
Algum motivo das aspas duplas?
There was a problem hiding this comment.
Just to follow a pattern. I prefer single quotes but almost all the file use double quotes.
Create a ConnectTimeoutError in correios lib to handle ConnectTimeout errors from requests lib.