Project

General

Profile

Développement #42729

Toujours retourner 200 dans la vue de callback

Added by Valentin Deniaud over 4 years ago. Updated over 4 years ago.

Status:
Rejeté
Priority:
Normal
Target version:
-
Start date:
11 May 2020
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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.


Files

Associated revisions

Revision 7202ecd0 (diff)
Added by Valentin Deniaud over 4 years ago

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

History

#1

Updated by Valentin Deniaud over 4 years ago

#2

Updated by Valentin Deniaud over 4 years ago

  • Description updated (diff)
#3

Updated by Benjamin Dauvergne over 4 years ago

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

Updated by Valentin Deniaud over 4 years ago

  • Status changed from Solution proposée to 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.

Also available in: Atom PDF