Development #25690
to_json : ne pas logguer du tout pour les endpoints modernes
0%
Description
D'une note dans #25689 : je pense qu'on doit arriver à un moment à logguer dans to_json() uniquement sur les connecteurs "legacy", que le logging correct se fasse via @endpoint.
Fichiers
Demandes liées
Historique
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Lié à Development #25689: to_json : utiliser le logger du connecteur ajouté
Mis à jour par Frédéric Péters il y a environ 4 ans
(et limite je serais prêt à ce qu'on dise ici tant pis pour les vieux endpoints, to_json ne loggue plus rien).
Mis à jour par Valentin Deniaud il y a environ 4 ans
Ce que demande le ticket c'est de rendre le code plus cohérent en déplaçant le logging de to_json vers endpoint, qui se trouve dans utils/api.py ?
Pourtant dans #25689 tu disais :
Ça révèle pas mal que le logging de to_json() est plutôt pourri...
Déplacer le code ne va pas le rendre plus ou moins pourri, donc j'aimerais comprendre si il y a quelque chose que je zappe avant de me lancer.
Mis à jour par Frédéric Péters il y a environ 4 ans
Non c'est totalement virer le code (de logging) qui est dans to_json qui m'intéresse.
Mis à jour par Lauréline Guérin il y a environ 4 ans
- Fichier 0001-misc-remove-the-use-of-to_json-decorator-for-modern-.patch 0001-misc-remove-the-use-of-to_json-decorator-for-modern-.patch ajouté
- Patch proposed changé de Non à Oui
à discuter :)
Mis à jour par Lauréline Guérin il y a environ 4 ans
- Statut changé de Nouveau à Solution proposée
Mis à jour par Lauréline Guérin il y a presque 4 ans
- Statut changé de Solution proposée à Rejeté
Après discussion avec Fred: on laisse tel quel.
Ticket trop vieux, nécessite un état des lieux plus approfondi, et un plan d'action.