Projet

Général

Profil

Bug #9351

Terminer la notification de paiement à wcs (cas local)

Ajouté par Frédéric Péters il y a plus de 8 ans. Mis à jour il y a presque 8 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
16 décembre 2015
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Pour le moment c'est bricolé avec une fausse clé…


Fichiers


Demandes liées

Lié à Hobo - Bug #8580: orig et secret dans KNOWN_SERVICESFermé09 octobre 2015

Actions

Révisions associées

Révision bb4ce38d (diff)
Ajouté par Frédéric Péters il y a plus de 8 ans

lingo: notify wcs about local items payment (#9351)

Historique

#1

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é
#2

Mis à jour par Serghei Mihai il y a plus de 8 ans

#3

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

Ç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.

#4

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

Le verif_orig apparait seulement avec #8580, modification pour le calculer.

#5

Mis à jour par Serghei Mihai il y a plus de 8 ans

Ok

#6

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)
#8

Mis à jour par Frédéric Péters il y a presque 8 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF