Development #25773
ajouter une explication lors des raise ResponseError
Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
22 août 2018
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
dans combo/lingo quand on reçoit des eopayment.ResponseError on fait un log de « 'failed to truc: %s' % exception »
mais comme eopayment ne fait que des « raise ResponseError() » le %s est juste remplacé par rien, et on juste un bête 'failed to truc:'
Il faudrait donner un peu de contexte lors des ResponseError, par exemple :
diff --git a/eopayment/systempayv2.py b/eopayment/systempayv2.py
index d2de01c..1e427f2 100644
--- a/eopayment/systempayv2.py
+++ b/eopayment/systempayv2.py
@@ -352,7 +352,7 @@ class Payment(PaymentCommon):
def response(self, query_string, **kwargs):
fields = urlparse.parse_qs(query_string, True)
if not set(fields) >= set([SIGNATURE, VADS_CTX_MODE, VADS_AUTH_RESULT]):
- raise ResponseError()
+ raise ResponseError('missing SIGNATURE/VADS_CTX_MODE/VADS_AUTH_RESULT fields in %s' % query_string)
for key, value in fields.items():
fields[key] = value[0]
copy = fields.copy()
Fichiers
Demandes liées
Révisions associées
debian: add missing python-pytest build dependency (#25773)
Historique
Mis à jour par Thomas Noël il y a plus de 5 ans
- Lié à Support #25772: Correctement loguer la réponse de paiement ajouté
Mis à jour par Thomas Noël il y a plus de 5 ans
- Fichier 0001-add-a-message-in-ResponseErrors-25773.patch 0001-add-a-message-in-ResponseErrors-25773.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
(stp pense à poser le statut sinon je rate les ack)
Mis à jour par Thomas Noël il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit e8f12c5d1fd7b9cf5bd2526898b2004a3bbee8b0
Author: Thomas NOEL <tnoel@entrouvert.com>
Date: Wed Aug 22 23:14:53 2018 +0200
add a message in ResponseErrors (#25773)
Mis à jour par Frédéric Péters il y a plus de 5 ans
Et aussi,
commit d142c1c8f82c995fd05cc6cbeff56ac4a94c0b74
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Wed Aug 29 12:05:04 2018 +0200
debian: add missing python-pytest build dependency (#25773)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
add a message in ResponseErrors (#25773)