Development #31089
webservice: toujours ignorer les soucis en cas d'erreur
0%
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").
Historique
Mis à jour par Frédéric Péters il y a environ 5 ans
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).
Mis à jour par Thomas Noël il y a environ 5 ans
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)