Projet

Général

Profil

Bug #39074

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

Ajouté par Benjamin Dauvergne il y a plus de 4 ans. Mis à jour il y a environ 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Emmanuel Cazenave
Version cible:
-
Début:
17 janvier 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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.

Fichiers

0001-lingo-move-transaction-identifier-to-URL-path-39074.patch (8,6 ko) 0001-lingo-move-transaction-identifier-to-URL-path-39074.patch Emmanuel Cazenave, 20 janvier 2020 17:59

Révisions associées

Révision 3177f506 (diff)
Ajouté par Emmanuel Cazenave il y a plus de 4 ans

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

Historique

#1

Mis à jour par Benjamin Dauvergne il y a plus de 4 ans

  • Description mis à jour (diff)
#2

Mis à jour par Emmanuel Cazenave il y a plus de 4 ans

  • Sujet changé de Paiement sans authentificatio ni panier (bis) à Passer lingo-transaction-id dans le chemin de l'URL plutôt qu'en paramètre
  • Assigné à mis à Emmanuel Cazenave
#3

Mis à jour par Emmanuel Cazenave il y a plus de 4 ans

#4

Mis à jour par Frédéric Péters il y a plus de 4 ans

  • Statut changé de Solution proposée à Solution validée
#5

Mis à jour par Emmanuel Cazenave il y a plus de 4 ans

  • Statut changé de Solution validée à 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

Mis à jour par Frédéric Péters il y a environ 4 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF