Produits Entr'ouvert » Passerelle

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)

Mon résumé des discussions plus haut : tout logger sauf les 404.
Idéalement il faudrait logger les 404 qui correspondent à une mauvaise URL, mais le retour de l'API ne permet pas de distinguer les deux (à moins d'inspecter le message d'erreur verbeux en français, mais on ne va pas faire ça).

(tests verts)

(j'avais oublié python2)

Valentin Deniaud a écrit :

Mon résumé des discussions plus haut : tout logger sauf les 404.
Idéalement il faudrait logger les 404 qui correspondent à une mauvaise URL, mais le retour de l'API ne permet pas de distinguer les deux (à moins d'inspecter le message d'erreur verbeux en français, mais on ne va pas faire ça).

Effectivement ça renvoie toujours du JSON :/
Par contre on peut parser un peu, sur une vrai 404 ça fait :

{
  "error": "not_found",
  "reason": "no route for URL /api/v23avis-imposition?3numeroFiscal=1902599999001&referenceAvis=1902599999001",
  "message": "no route for URL /api/v23avis-imposition?3numeroFiscal=1902599999001&referenceAvis=1902599999001" 
}

un simple 'incorrects ou ne correspondent' in response.json['reason'] devrait suffire à séparer les deux, s'ils changent de chaîne (ce qu'ils font assez rarement) on aura des erreurs, on corrigera la chaîne.

Aussi je renverrai la totalité du retour au cas où plutôt que juste la clé message avec APIError(data={'code': 'not-found', 'response': response.json}) dans tous les cas avec un code d'erreur à nous (non-json, non-200, connection-error, etc..).

Benjamin Dauvergne a écrit :

un simple 'incorrects ou ne correspondent' in response.json['reason'] devrait suffire à séparer les deux, s'ils changent de chaîne (ce qu'ils font assez rarement) on aura des erreurs, on corrigera la chaîne.

Ça ne me semble pas possible de faire ça car en ce moment l'endpoint CAF est cassé, on ne sait pas si le message d'erreur sera celui-là. On peut éventuellement adopter la logique inverse, 'no route' in ..., m'enfin j'étais bien parti pour ne rien faire alors je te laisse insister.

Aussi je renverrai la totalité du retour au cas où plutôt que juste la clé message avec APIError(data={'code': 'not-found', 'response': response.json}) dans tous les cas avec un code d'erreur à nous (non-json, non-200, connection-error, etc..).

Done, dans un nouveau patch.

commit e0207afef5d8b9f8147b8db584e3cefaa42c7d6d
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Wed Jun 3 16:21:08 2020 +0200

    api_particulier: improve api errors (#38721)

commit a7980a2a701d48067d22a894763e606e9e46e4d7
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Wed Jun 3 10:59:13 2020 +0200

    api_particulier: do not log 404 as errors (#38721)

commit 859385cf55fd8910d9595c7c43d89edcbea4162e
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Wed Jun 3 10:54:27 2020 +0200

    utils: allow changing APIError attributes (#38721)