Project

General

Profile

Bug #38721

api_particulier: ne pas lever les erreurs des requetes HTTP

Added by Serghei Mihai about 2 months ago. Updated 8 days ago.

Status:
En cours
Priority:
Normal
Assignee:
Target version:
-
Start date:
30 Dec 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Pour éviter les notifications des erreurs 404, comme par exemple: https://particulier.api.gouv.fr/api/impots/svair?numeroFiscal=3025407412230&referenceAvis=19+13+M521419

0001-api_particulier-don-t-log-http-errers-38721.patch View (786 Bytes) Serghei Mihai, 30 Dec 2019 12:50 PM

History

#1 Updated by Serghei Mihai about 2 months ago

#2 Updated by Thomas Noël about 2 months ago

Sur l'URL j'ai une 401 avec un contenu qui n'est pas du JSON... je pense que ça doit être un soucis de token, rare, et qu'une alerte est utile dans ce cas.

#3 Updated by Thomas Noël 11 days ago

  • Status changed from Solution proposée to En cours

Est-ce que la 404 indiquée en haut est une réponse normale d'api particulier qui veut dire "je ne connais pas ce numéro fiscal" ? ie encore une API qui utilise le transport http pour faire passer de l'info ?

Auquel cas on peut effectivement ne pas lever les erreurs des requetes HTTP, mais il faudrait quand même lever quelque chose si ce n'est pas 404 (et en cas de 404, renvoyer une APIError)

#4 Updated by Benjamin Dauvergne 11 days ago

Oui 404 correspond à une réponse applicative pas de donnée ou on s'est trompé d'URL :/

404 Les paramètres fournis sont incorrects ou ne correspondent pas à un avis

Mais dans le cas d'une erreur applicative on a un JSON avec, c'est ça qu'il faut tester, si JSON -> applicatif si pas JSON, transport.

Donc ça suppose log_request_errors = False mais aussi de traiter les retours adéquatement (comme Sheila).

#5 Updated by Thomas Noël 11 days ago

Benjamin Dauvergne a écrit :

si JSON -> applicatif si pas JSON, transport.

Encore plus efficace, yep.

#6 Updated by Benjamin Dauvergne 8 days ago

  • Assignee set to Serghei Mihai

Also available in: Atom PDF