Bug #9351
Terminer la notification de paiement à wcs (cas local)
0%
Description
Pour le moment c'est bricolé avec une fausse clé…
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 8 ans
- Lié à Bug #8580: orig et secret dans KNOWN_SERVICES ajouté
Mis à jour par Serghei Mihai il y a plus de 8 ans
- Fichier 0001-lingo-notify-wcs-about-local-items-payment-9351.patch 0001-lingo-notify-wcs-about-local-items-payment-9351.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus de 8 ans
- Fichier 0001-lingo-notify-wcs-about-local-items-payment-9351.patch 0001-lingo-notify-wcs-about-local-items-payment-9351.patch ajouté
Ça n'avait jamais même été exécuté.
for service in settings.KNOWN_SERVICES['combo']: if service['orig'] == request.get_host():
(cette boucle donne les clés dans "service", pas les valeurs → request['orig] foire sur un "TypeError: string indices must be integers, not str".
Ensuite, pour signer la requête, je sais que c'est toujours confus mais c'est le service qu'on appelle qu'il faut chercher, pour avoir la bonne clé.
Aussi alors qu'avant il y avait un faux email (trigger@localhost), c'est maintenant remplacé par l'email de l'utilisateur associé au paiement. Ça veut dire que l'usager peut aller sur le wcs faire le trigger manuellement et faire passer le paiement comme étant payé. On ne peut pas accepter ça. Ma proposition serait de ne pas inclure d'email et de modifier la gestion côté wcs pour, en l'absence d'email, et de restriction de rôle, accepter les requêtes signées.
Mis à jour par Frédéric Péters il y a plus de 8 ans
- Fichier 0001-lingo-notify-wcs-about-local-items-payment-9351.patch 0001-lingo-notify-wcs-about-local-items-payment-9351.patch ajouté
Le verif_orig apparait seulement avec #8580, modification pour le calculer.
Mis à jour par Frédéric Péters il y a plus de 8 ans
- Statut changé de En cours à Résolu (à déployer)
commit bb4ce38dce91857872534b603d2fb1da54cc2ad2 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Sun Jan 10 15:54:39 2016 +0100 lingo: notify wcs about local items payment (#9351)
lingo: notify wcs about local items payment (#9351)