Projet

Général

Profil

Development #12397

Ne pas stopper le workflow en cas d'erreur connue du webservice

Ajouté par Victor Claudet il y a presque 8 ans. Mis à jour il y a plus de 7 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
01 juillet 2016
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:

Description

sold out est considéré comme une erreur et stop le workflow, sauf que nous devons pouvoir réagir face à ce type d'erreur, il faut donc soit considérer que soldout n'est pas une erreur, soit pouvoir contrôler cette erreur au niveau de l'appel webservice et choisir comme les autres types d'erreur (ignorer, arreter, changer de statut...)

Révisions associées

Révision e27c3158 (diff)
Ajouté par Frédéric Péters il y a presque 8 ans

api: don't use 400 error code for "sold out" error (#12397)

Historique

#1

Mis à jour par Victor Claudet il y a presque 8 ans

en réalité c'est déjà possible puisque ce type d'erreur est considéré comme une erreur 400 (et on peut gérer les erreurs 4XX dans l'appel webservice), mais est-ce que c'est pas dangereux de considérer un même fonctionnement dans le cas d'une 400 que d'une 404 par exemple ?

#2

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

On ne devrait pas utiliser un 4XX pour une situation qui émane du serveur et pas de la requête, faut soit un 200 soit un 500 avec un code d'erreur.

#3

Mis à jour par Frédéric Péters il y a presque 8 ans

Sur le cas précis c'est considéré comme une mauvaise requête sur l'idée que passer une date invalide est une erreur de la requête.

#4

Mis à jour par Victor Claudet il y a presque 8 ans

c'est réellement un problème dans la gestion du workflow derrière.
Le webservice répond une erreur semblable à une erreur technique, alors que c'est une réponse "normale". du coup impossible de gérer proprement le cas des réservations complètes dans le workflow.

#5

Mis à jour par Frédéric Péters il y a presque 8 ans

ok ok; tu aurais une URL pour formulaire/workflow directement concerné ?

#6

Mis à jour par Frédéric Péters il y a presque 8 ans

  • Statut changé de Nouveau à Résolu (à déployer)
commit e27c31583caf033b26591e82a28d10d385fb5ec2
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed Jul 20 08:48:56 2016 +0200

    api: don't use 400 error code for "sold out" error (#12397)
#8

Mis à jour par Frédéric Péters il y a plus de 7 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF