Project

General

Profile

Development #31089

webservice: toujours ignorer les soucis en cas d'erreur

Added by Thomas Noël 3 months ago. Updated 3 months ago.

Status:
Nouveau
Priority:
Bas
Assignee:
-
Start date:
05 Mar 2019
Due date:
% Done:

0%

Patch proposed:
No
Planning:
No

Description

Actuellement, historiquement, on a ces différents comportements sur un appel webservice :

    action_on_app_error = ':pass'
    action_on_4xx = ':stop'
    action_on_5xx = ':stop'
    action_on_bad_data = ':pass'
    action_on_network_errors = ':stop'

J'aimerai que ça soit ":pass" partout, et donc retrouver la raison initiale de cette disparité.

Et au passage renommer ":pass" en "Ignore & continue" (au lieu de juste "Ignore").

History

#1 Updated by Frédéric Péters 3 months ago

J'aimerai que ça soit ":pass" partout, et donc retrouver la raison initiale de cette disparité.

Parce qu'au moment où ça été rendu configurable ça a repris le comportement qui existait. Et app_error et bad_data sont arrivées plus tard.

Je ne suis vraiment pas fan de passer ça en :pass, ça va décaler des erreurs qui seront encore moins compréhensibles, ça rendra aussi l'affaire plus compliquée à débugguer/remettre droit (aujourd'hui facilement on ajoute un saut "replay" dans l'action et ça repart).

#2 Updated by Thomas Noël 3 months ago

Frédéric Péters a écrit :

J'aimerai que ça soit ":pass" partout, et donc retrouver la raison initiale de cette disparité.

Parce qu'au moment où ça été rendu configurable ça a repris le comportement qui existait. Et app_error et bad_data sont arrivées plus tard.

On pourrait donc "faire le tour" des wscall et fixer le :stop quand il n'est pas défini, ie sur de vieux workflow jamais ré-enregistrés ?

Je ne suis vraiment pas fan de passer ça en :pass, ça va décaler des erreurs qui seront encore moins compréhensibles, ça rendra aussi l'affaire plus compliquée à débugguer/remettre droit (aujourd'hui facilement on ajoute un saut "replay" dans l'action et ça repart).

Ouaip.

Mais ":stop" c'est pas mieux, pour ne pas dire pire. S'arrêter en plein milieu d'un statut, on n'a rien pour "reprendre le cours des opérations" ensuite ; wscall est la seule action qui agisse de la sorte, je crois.

On peut aussi laisser ainsi, mais je n'aime pas beaucoup avoir ces comportements différents, et le ":pass" obligerait à un peu plus d'attention. (globalement je constate que personne ne fait trop attention à cette partie de la configuration d'un appel webservice, et c'est un peu dommage, et ce ticket n'est peut-être pas la façon la plus rusée de forcer l'attention)

#3 Updated by Thomas Noël 3 months ago

  • Priority changed from Normal to Bas

Also available in: Atom PDF