Project

General

Profile

Bug #39074

Passer lingo-transaction-id dans le chemin de l'URL plutôt qu'en paramètre

Added by Benjamin Dauvergne about 1 month ago. Updated 28 days ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
17 Jan 2020
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Je viens de constater un souci avec le backend dummy avec les derniers développement, comme on passe l'id de transaction dans la query-string de payment status view et que certains backend retourne aussi des choses en query-string, il faut que le backend gère les URLs contenant déjà une query-string.

Ce n'était pas le cas de notre brave écureuil et j'ai corrigé ça, néanmoins si ça arrive avec un vrai backend on aura pas les moyens de corriger.

Je proposerai que lingo-transaction-id soit passé dans le chemin de l'URL et non pas en paramètre, ça nous évitera ce possible souci.

Pour dummy ça donnait cette URL de retour :

(15:48:44) Laureline Guérin: "GET /lingo/return-payment-backend/1/?lingo-transaction-id=Mzg:1isSe9:OdU7jf01Hb7s5qqXtLvkrFa7fFM?transaction_id=cDSXCuyhiNa8ngH8blWFh1xWC1xlVM&siret=siret_num&amount=75.57&email=admin%40localhost&return_url=https%3A%2F%2Fcombo.dev.publik.love%2Flingo%2Freturn-payment-backend%2F1%2F%3Flingo-transaction-id%3DMzg%3A1isSe9%3AOdU7jf01Hb7s5qqXtLvkrFa7fFM&direct_notification_url=https%3A%2F%2Fcombo.dev.publik.love%2Flingo%2Fcallback-payment-backend%2F1%2F&ok=1&signed=1 HTTP/1.0" 500 92719

On voit que transaction_id venant de l'écureuil est bêtement ajouté derrière l'URL.

0001-lingo-move-transaction-identifier-to-URL-path-39074.patch View (8.6 KB) Emmanuel Cazenave, 20 Jan 2020 05:59 PM

Associated revisions

Revision 3177f506 (diff)
Added by Emmanuel Cazenave about 1 month ago

lingo: move transaction identifier to URL path (#39074)

History

#1 Updated by Benjamin Dauvergne about 1 month ago

  • Description updated (diff)

#2 Updated by Emmanuel Cazenave about 1 month ago

  • Assignee set to Emmanuel Cazenave
  • Subject changed from Paiement sans authentificatio ni panier (bis) to Passer lingo-transaction-id dans le chemin de l'URL plutôt qu'en paramètre

#3 Updated by Emmanuel Cazenave about 1 month ago

#4 Updated by Frédéric Péters about 1 month ago

  • Status changed from Solution proposée to Solution validée

#5 Updated by Emmanuel Cazenave about 1 month ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 3177f5067dcad501bcce438a2918e3aa5747e10a
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date:   Mon Jan 20 17:59:01 2020 +0100

    lingo: move transaction identifier to URL path (#39074)

#6 Updated by Frédéric Péters 28 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF