Development #53407
connecteur liferay/strasbourg.eu, gérer leurs erreurs
0%
Description
Sur un POST https://***:***@mon.strasbourg.eu/api/jsonws/notification.notification/get-user-notifications qui donne une erreur 500 mais du contenu json valide ({}), on enchaine en trébuchant dessus, on devrait retourner l'info comme quoi il y a eu erreur.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 3 ans
- Fichier 0001-strasbourg_eu-handle-liferay-error-500-but-valid-JSO.patch 0001-strasbourg_eu-handle-liferay-error-500-but-valid-JSO.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a environ 3 ans
Juste pour savoir, quand il y a un pépin leur webservice répond une 200 avec un json contenant un "error" ? (si oui c'est bien, mais comme c'est rare, je demande, parce qu'on ne les remontera plus de la même façon).
Mis à jour par Frédéric Péters il y a environ 3 ans
quand il y a un pépin
Ça dépend du pépin; formellement c'était ça et je n'avais pas encore vu de leur part erreur 500 + json valide/vide.
Pour moi le comportement face à une 2OO + clé error n'est pas modifié par le patch, uniquement l'hypothétique erreur HTTP + clé error, qui est une situation pas encore rencontrée; mais vu que je l'ai écrit je passe peut-être à côté de ce que tu vois.
Mis à jour par Thomas Noël il y a environ 3 ans
- Statut changé de Solution proposée à Solution validée
Frédéric Péters a écrit :
quand il y a un pépin
Ça dépend du pépin; formellement c'était ça et je n'avais pas encore vu de leur part erreur 500 + json valide/vide.
Pour moi le comportement face à une 2OO + clé error n'est pas modifié par le patch, uniquement l'hypothétique erreur HTTP + clé error, qui est une situation pas encore rencontrée ; mais vu que je l'ai écrit je passe peut-être à côté de ce que tu vois.
Je ne connais pas la réalité de leur webservice, mais d'expérience on voit souvent des webservices renvoyer un JSON avec "error": "inexistant" mais avec aussi un statut 404...
Mais je comprend qu'on est face à un webservice mieux géré qui ne confond pas transport et contenu, et qui renvoie bien ses "error" dans des 200 et oui alors le patch ne modifie rien et il est tout bon.
Je valide donc, désolé pour le bruit.
Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 60b77ad5da9a609e8431c462c080f046ee6da047
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sun Apr 25 11:32:18 2021 +0200
strasbourg_eu: handle liferay error 500 (but valid JSON) answers (#53407)
Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
strasbourg_eu: handle liferay error 500 (but valid JSON) answers (#53407)