Bug #42659
api particulier, 'NoneType' object has no attribute 'status_code' lors de timeout de l'api
0%
Description
https://sentry.entrouvert.org/entrouvert/publik/issues/2063/
AttributeError: 'NoneType' object has no attribute 'status_code' File "passerelle/utils/jsonresponse.py", line 128, in api resp = f(*args, **kwargs) File "passerelle/views.py", line 475, in perform result = self.endpoint(request, **params) File "passerelle/apps/api_particulier/models.py", line 144, in impots_svair }, user=user) File "passerelle/apps/api_particulier/models.py", line 85, in get (self.platform, response.status_code), Error occurred while processing request
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 4 ans
- Projet changé de Suivi des traces à Passerelle
- Sujet changé de AttributeError: 'NoneType' object has no attribute 'status_code' à api particulier, 'NoneType' object has no attribute 'status_code' lors de timeout de l'api
Mis à jour par Lauréline Guérin il y a presque 4 ans
- Fichier 0001-api_particulier-fix-connection-error-42659.patch 0001-api_particulier-fix-connection-error-42659.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Nicolas Roche il y a presque 4 ans
Juste pour comprendre, pourquoi tu ne pointe pas sur conversions.py pour importer exception_to_text ?
from passerelle.utils.jsonresponse import APIError, exception_to_text
Mis à jour par Lauréline Guérin il y a presque 4 ans
Mis à jour par Valentin Deniaud il y a presque 4 ans
- Statut changé de Solution proposée à Solution validée
Juste que je ne toucherais pas au paramètre data de l'APIError, peut-être qu'à un endroit il y a un wcs qui regarde ce qu'il y a dans response_data_data_platform, qui sait.
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
Valentin Deniaud a écrit :
Juste que je ne toucherais pas au paramètre data de l'APIError, peut-être qu'à un endroit il y a un wcs qui regarde ce qu'il y a dans response_data_data_platform, qui sait.
Pareil je ne vois pas le besoin de diminuer l'information disponible, à la rigueur l’accroître avec un code stable utilisable dans des conditions ça j'aime bien.
Mis à jour par Valentin Deniaud il y a presque 4 ans
Benjamin Dauvergne a écrit :
à la rigueur l’accroître avec un code stable utilisable dans des conditions ça j'aime bien.
Il y a déjà un patch qui fait ça dans #38721, pas besoin d'y toucher ici.
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
Valentin Deniaud a écrit :
Il y a déjà un patch qui fait ça dans #38721, pas besoin d'y toucher ici.
Oublié, donc reste juste à ne pas perdre le platform au passage.
Mis à jour par Lauréline Guérin il y a presque 4 ans
- Fichier 0001-api_particulier-fix-connection-error-42659.patch 0001-api_particulier-fix-connection-error-42659.patch ajouté
- Statut changé de Solution validée à Solution proposée
J'avais copié/collé la façon de faire dans api_entreprise. Mauvaise idée apparemment :)
(note: j'ai supprimé la redéfinition de vector
dans le test test_error
, c'est les mêmes valeurs à chaque fois)
Mis à jour par Valentin Deniaud il y a presque 4 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Lauréline Guérin il y a presque 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 2ff16b6105780ac6774dc962e9b4c038192e0208 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Mon May 11 09:21:48 2020 +0200 api_particulier: fix connection error (#42659)
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
api_particulier: fix connection error (#42659)