Development #64468
traces sur vérif de is_notifiable
Début:
22 avril 2022
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
À l'origine #64460#note-2,
def is_notifiable(self, status='paid'):
if not self.source_url:
return True
url = self.source_url + 'jump/trigger/%s' % status
req = requests.get(url, remote_service='auto', timeout=5)
if req.status_code == 404:
return False
return True
Grosse frayeur mais en fait le .get() ne déclenchera pas le trigger.
Je serais pour 1/ ajouter un commentaire explicite au code, 2/ ne pas envoyer de trace à l'admin quand on tombe sur une 404.
Fichiers
Historique
Mis à jour par Frédéric Péters il y a presque 2 ans
- Fichier 0001-lingo-comment-is_notifiable-and-do-not-log-non-error.patch 0001-lingo-comment-is_notifiable-and-do-not-log-non-error.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Serghei Mihai il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
(et j'ai envoyé la version du patch avec le mauvais numéro de ticket)
commit 9dba7d12451cea2c78270991c45a92d49f30cfc1
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Fri Apr 22 08:44:07 2022 +0200
lingo: comment is_notifiable() and do not log non-errors (#64460)
Mis à jour par Frédéric Péters il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée