Development #44054
sources de données, contrôle de la notification des erreurs
0%
Description
Après #44050 il se révèle en fait que l'erreur initiale était au niveau d'une source de données, celles-ci devraient donc également pouvoir notifier les erreurs.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Sujet changé de sources de données, notifier les erreurs à sources de données, contrôle de la notification des erreurs
(ou ne pas notifier, parce qu'on a certains cas comme l'autocomplétion qui provoquent actuellement des traces en cas d'erreur de la source de données, je pense)
Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de Nouveau à En cours
- Assigné à mis à Frédéric Péters
Mis à jour par Frédéric Péters il y a environ 3 ans
- Fichier 0001-data-sources-add-error-management-parameters-44054.patch 0001-data-sources-add-error-management-parameters-44054.patch ajouté
- Statut changé de En cours à Solution proposée
- Patch proposed changé de Non à Oui
À False par défaut parce que la partie logguée via get_logger().warning(...) était quand même assez silencieuse jusqu'à présent.
Un bout à noter (et c'est en soit ce qui m'a relancé sur ce ticket), c'est l'endpoint d'autocomplétion, qui n'avait pas de gestion d'erreur et laissait du coup s'envoyer des traces.
Aussi, les LoggedErrors sont aujourd'hui uniquement enregistrées si elles peuvent être attachées à un objet, comme on a désormais une vue globale pour celles-ci, c'est quelque chose qui pourrait être revu.
Et la partie raise artificiel pour taper un message dans le sujet des messages est assez nulle et pourrait également évoluer, peut-être que notify_of_exception pourrait prendre un paramètre supplémentaire pour forcer un sujet. (mais j'ai préféré ne pas m'engager là-dedans ici).
Mis à jour par Lauréline Guérin il y a environ 3 ans
- Statut changé de Solution proposée à Solution validée
La partie record_on_errors n'est pas testée parce qu'actuellement ça ne génère pas de LoggedError, car non attaché à un formdef ou workflow ?
Pourrais-tu créer un ticket pour changer ça du coup ?
Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
Oui, c'est pour ça. (j'étais d'ailleurs parti sur loggederror pour les tests avant de m'en rendre compte). J'ai créé #52045.
commit 61367b05dc0ad22ff725464a6f158b69fb9c40e1
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sun Mar 14 12:24:50 2021 +0100
data sources: add error management parameters (#44054)
Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
data sources: add error management parameters (#44054)