Projet

Général

Profil

Development #42729

Toujours retourner 200 dans la vue de callback

Ajouté par Valentin Deniaud il y a presque 4 ans. Mis à jour il y a presque 4 ans.

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

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Vu dans la doc de Mollie, si on appelle la vue avec un ID inconnu elle ne devrait pas retourner d'erreur.
Possible de faire à la pince à épiler en ne rattrapant qu'une nouvelle eopayment.UnknownIDException et en la levant au bon moment, mais sauf avis contraire je m'évite ça et je rattrape tout avec log.


Fichiers

Révisions associées

Révision 7202ecd0 (diff)
Ajouté par Valentin Deniaud il y a presque 4 ans

lingo: always return 200 OK in CallbackView (#42729)

Historique

#1

Mis à jour par Valentin Deniaud il y a presque 4 ans

#2

Mis à jour par Valentin Deniaud il y a presque 4 ans

  • Description mis à jour (diff)
#3

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

Pour certains autres backends si on ne renvoie pas une erreur on aura jamais de rejeu (je pense à TIPI / PayFiP) ; donc il faudrait que tu te restreignes à l'erreur qui t'intéresse vraiment et peut-être uniquement dans le cas de mollie.

En dehors de la doc de mollie qui dit ne pas le faire qu'est-ce que ça fait exactement pour mollie ? Parce que dans l'absolu si ça ne casse pas mollie je verrai plutôt le ticket rejeté.

#4

Mis à jour par Valentin Deniaud il y a presque 4 ans

  • Statut changé de Solution proposée à Rejeté

Je me disais que si il y avait une erreur dans Payment.response() ça n'allait pas servir à grand chose de rejouer parce que le code est déterministe, mais en fait il peut y avoir une erreur réseau (dans payfip_ws et mollie, tipi je ne vois pas).

Et non ça ne casse rien, hop aux oubliettes.

Formats disponibles : Atom PDF