Project

General

Profile

Development #56647

remonter en erreur un callback vers une régie qui n'existe pas

Added by Frédéric Péters about 1 month ago. Updated 11 days ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
04 Sep 2021
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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.


Files

Associated revisions

Revision 14b4adc3 (diff)
Added by Lauréline Guérin 16 days ago

lingo: change log level when payment backend is not found (#56647)

History

#1

Updated by Lauréline Guerin 16 days ago

  • Assignee set to Lauréline Guerin
#2

Updated by Benjamin Dauvergne 16 days ago

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

#3

Updated by Lauréline Guerin 16 days ago

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 ?

#4

Updated by Lauréline Guerin 16 days ago

(juste le log level)

#5

Updated by Benjamin Dauvergne 16 days ago

  • Status changed from Solution proposée to 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.

#6

Updated by Lauréline Guerin 16 days ago

  • Status changed from Solution validée to 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)
#7

Updated by Frédéric Péters 11 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF