Projet

Général

Profil

Development #42512

lingo: renvoyer du JSON en cas d'erreur

Ajouté par Benjamin Dauvergne il y a presque 4 ans. Mis à jour il y a presque 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
05 mai 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Sur #42509 j'ai eu un peu de mal à trouver la cause de l'erreur parce que les 400 + text ne sont pas conservées par w.c.s., il faudrait remplacer toutes ces 400, renvoyée notamment par AddBasketItem par des APIError à la sauce passerelle.


Fichiers

Révisions associées

Révision 27236045 (diff)
Ajouté par Lauréline Guérin il y a presque 4 ans

lingo: return json error response (#42512)

Historique

#1

Mis à jour par Benjamin Dauvergne il y a presque 4 ans

  • Projet changé de Combo à Lingo
#2

Mis à jour par Lauréline Guérin il y a presque 4 ans

  • Assigné à mis à Lauréline Guérin
#3

Mis à jour par Lauréline Guérin il y a presque 4 ans

#4

Mis à jour par Benjamin Dauvergne il y a presque 4 ans

Plutôt BadRequestJsonResponse et status=400 par défaut ? et virer les quelques majuscules dans les messages ça fait sale.

J'allais proposer d'avoir des codes plus précis dans err mais comme ce ne sont vraiment que des erreurs d'utilisation qui ne doivent pas arriver ça ne me parait pas très utile, car on ne voudra jamais vraiment les gérer du coté de l'appelant; si tu en vois un qui aurait un sens au niveau fonctionnel ça pourrait être utile de pouvoir le différencier des autres avec un err='<error-code>'.

#6

Mis à jour par Benjamin Dauvergne il y a presque 4 ans

  • Statut changé de Solution proposée à Solution validée
#7

Mis à jour par Lauréline Guérin il y a presque 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 2723604596a79f997fee4407fad0211f983555f9
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date:   Thu May 14 17:06:26 2020 +0200

    lingo: return json error response (#42512)
#8

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

Formats disponibles : Atom PDF