Development #56647
remonter en erreur un callback vers une régie qui n'existe pas
0%
Description
Un POST d'une plateforme de paiement vers une URL genre /lingo/callback-payment-backend/34/ va inscrire dans les logs
Sep 4 13:46:30 combo uwsgi[19609]: combo WARNING roanne.icitoyen.fr 194.50.38.6 - r:7FE45157F7B8 lingo: payment backend not found on callback kwargs={'payment_backend_pk': '38'}
Ça passera assez inaperçu alors que c'est plutôt signe d'une mauvaise configuration de la plateforme de paiement, ce qui devrait amener quelque chose de plus visible.
Monter au niveau error, qui ferait qu'on reçoive un email, serait le minimum mais il y a peut-être à lancer aussi une réflexion sur la visibilité de ces erreurs aux admins fonctionnels en charge de la gestion des paiements.
Fichiers
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
Ça demande peut-être un mécanisme un peu semblable aux LoggerError de w.c.s. dans ce cas, ça peut commencer assez simple avec un :
class Alert: level = WARNING / ERROR first_seen = ... last_seen = ... count = IntegerField(...) object_ct = ... object_id = ... object = GenericForeignKey(...) message = ....
et passer par un handler de log tout con qui cherche si il y a un objet Django dans record.object. Je sais qu'il y aussi un système pour les erreurs sur les cellules peut-être que c'est réutilisable aussi (je n'ai pas le modèle en tête).
Mis à jour par Lauréline Guérin il y a plus de 2 ans
est-ce qu'on commence ce chantier dans ce ticket, ou on le cantonne à changer le niveau de log et on ouvre un autre ticket ?
Mis à jour par Lauréline Guérin il y a plus de 2 ans
- Fichier 0001-lingo-change-log-level-when-payment-backend-is-not-f.patch 0001-lingo-change-log-level-when-payment-backend-is-not-f.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
(juste le log level)
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Je laisse les gens qui bossent plus souvent sur combo décider du bien fondé d'aller plus loin dans un autre ticket, c'était juste une idée.
Mis à jour par Lauréline Guérin il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 14b4adc38cf877c7d89042fb994fa0ac640cdedb Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Oct 1 10:07:25 2021 +0200 lingo: change log level when payment backend is not found (#56647)
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
lingo: change log level when payment backend is not found (#56647)