Development #61401
En cas d'erreur sur le retour de paiement, logger le contenu reçu
Début:
03 février 2022
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Cf. #51801 où il est tout à fait difficile de débugger ce qui se passe sans ces informations.
Fichiers
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a environ 2 ans
- Fichier 0002-lingo-always-log-notification-content-on-errors-6140.patch 0002-lingo-always-log-notification-content-on-errors-6140.patch ajouté
- Fichier 0001-lingo-ignore-empty-response-in-return-view-61240.patch 0001-lingo-ignore-empty-response-in-return-view-61240.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Construit au dessus de #61240 (à pousser avant donc).
Mis à jour par Frédéric Péters il y a environ 2 ans
- Statut changé de Solution proposée à En cours
Mis à jour par Benjamin Dauvergne il y a environ 2 ans
- Fichier 0001-lingo-always-log-notification-content-on-errors-6140.patch 0001-lingo-always-log-notification-content-on-errors-6140.patch ajouté
- Statut changé de En cours à Solution proposée
Un peu de code churn pour arrêter de mélanger eopayment.PaymentException et lingo.models.PaymentException.
Mis à jour par Benjamin Dauvergne il y a environ 2 ans
- Fichier 0001-lingo-always-log-notification-content-on-errors-6140.patch 0001-lingo-always-log-notification-content-on-errors-6140.patch ajouté
Pour faire plaisir à pylint.
Mis à jour par Benjamin Dauvergne il y a environ 2 ans
- Fichier 0001-lingo-always-log-notification-content-on-errors-6140.patch 0001-lingo-always-log-notification-content-on-errors-6140.patch ajouté
Aussi:
- le retour pour une notification d'un paiement inconnu est devenu une 400 c'est plus logique finalement, l'URL du endpoint existe bien.
- déplacement des logs "pas exceptionnels" vers les vues, elles sont plus à même de voir ce qu'il faut faire
Mis à jour par Nicolas Roche il y a environ 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Benjamin Dauvergne il y a environ 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 5accf65139b5add872eb11c4e7474d94fa756014
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Thu Feb 3 16:50:30 2022 +0100
lingo: always log notification content on errors (#61401)
- rename models.PaymentException to LingoException to prevent confusion
- attach transaction to exceptions when possible
- log in views instead of model methods, it's a better point to make
decisions on loggingMis à jour par Transition automatique il y a environ 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
lingo: always log notification content on errors (#61401)
- rename models.PaymentException to LingoException to prevent confusion
- attach transaction to exceptions when possible
- log in views instead of model methods, it's a better point to make
decisions on logging