Development #61292
retirer (un max de) l'ancien système de logging
Début:
31 janvier 2022
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Fichiers
Révisions associées
general: remove tracing logs (#61292)
logger: remove user/session tracing support (#61292)
workflows: remove debug logs from geolocate action (#61292)
portfolio: don't log successful pushes (#61292)
misc: do not log condition errors (#61292)
misc: log to stderr (-> journald), remove storing of log file (#61292)
misc: move bot user agent check into request objects (#61292)
trivial: remove leftover link to logs (#61292)
Historique
Mis à jour par Frédéric Péters il y a environ 2 ans
- Fichier 0008-misc-move-bot-user-agent-check-into-request-objects-.patch 0008-misc-move-bot-user-agent-check-into-request-objects-.patch ajouté
- Fichier 0007-misc-log-to-stderr-journald-remove-storing-of-log-fi.patch 0007-misc-log-to-stderr-journald-remove-storing-of-log-fi.patch ajouté
- Fichier 0006-misc-do-not-log-condition-errors-61292.patch 0006-misc-do-not-log-condition-errors-61292.patch ajouté
- Fichier 0005-portfolio-don-t-log-successful-pushes-61292.patch 0005-portfolio-don-t-log-successful-pushes-61292.patch ajouté
- Fichier 0004-workflows-remove-debug-logs-from-geolocate-action-61.patch 0004-workflows-remove-debug-logs-from-geolocate-action-61.patch ajouté
- Fichier 0003-logger-remove-user-session-tracing-support-61292.patch 0003-logger-remove-user-session-tracing-support-61292.patch ajouté
- Fichier 0002-general-remove-tracing-logs-61292.patch 0002-general-remove-tracing-logs-61292.patch ajouté
- Fichier 0001-general-reduce-logging-infrastructure-do-not-expose-.patch 0001-general-reduce-logging-infrastructure-do-not-expose-.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
0001 pour virer les pages de paramétrage et d'affichage des logs de l'UI, puis une succession de petits commits ciblés,
- 0002 pour retirer le logging de traçage des actions usager mis en place il y a très très longtemps,
- 0003 pour retirer l'infra qui permettait ces traces (log user id et session id)
- 0004 pour virer du logging de l'action de géocodage (les vraies erreurs possibles passent déjà dans record_error)
- 0005 pour virer le log quand l'action d'envoi dans le porte-doc réussit (c'est normal)
- 0006 pour virer la possibilité de logguer les erreurs sur les conditions, c'est historique sur les conditions de page, souvent écrites avec des dépendances sur des variables pas encore présentes, c'est aussi beaucoup moins pertinent aujourd'hui avec des gabarits django. (dans les logs encore activés j'en vois juste sur deux démarches)
- 0007 pour modifier l'infra de logging qui reste pour sortir ça sur stderr (StreamHandler), avec un formatter qui reprend le tenant et pas de date/heure, pour être utile dans la sortie journald)
- 0008 pour déplacer un bout de vérification de user agent de bot qui était dans logger.py mais utilisé exclusivement depuis http_requests.py.
Ça laisse des possibilités dans un temps à venir d'aller vers des bouts communs dans Hobo mais ça sera à réfléchir plus tard.
Mis à jour par Emmanuel Cazenave il y a environ 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a environ 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit a447aa18f41899cc894de030c4d8dd98bdd293f2 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Tue Feb 1 14:29:37 2022 +0100 misc: move bot user agent check into request objects (#61292) commit a3ae79b506ec3ccac46378e28a1ce7bce437e8ad Author: Frédéric Péters <fpeters@entrouvert.com> Date: Tue Feb 1 09:26:04 2022 +0100 misc: log to stderr (-> journald), remove storing of log file (#61292) commit 731f6881c799ce20cc76e41efc72b875a0ae19a6 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Mon Jan 31 19:57:27 2022 +0100 misc: do not log condition errors (#61292) commit b0eaa69785ec978be4f9c508148e582fe79f9e8a Author: Frédéric Péters <fpeters@entrouvert.com> Date: Mon Jan 31 19:54:42 2022 +0100 portfolio: don't log successful pushes (#61292) commit db4ad776520dc7f857477c2e6ace2b159bf079bf Author: Frédéric Péters <fpeters@entrouvert.com> Date: Mon Jan 31 19:53:15 2022 +0100 workflows: remove debug logs from geolocate action (#61292) commit 651008deaea1361c2e255b4cfce48f3755053731 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Mon Jan 31 20:05:22 2022 +0100 logger: remove user/session tracing support (#61292) commit 080ec01cced6ceb5abf93795e7eb02c2a4d54a65 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Mon Jan 31 19:39:01 2022 +0100 general: remove tracing logs (#61292) commit 9d39ded12540c1b8a65d625a173cd5f188808aab Author: Frédéric Péters <fpeters@entrouvert.com> Date: Mon Jan 31 19:34:52 2022 +0100 general: reduce logging infrastructure, do not expose anything in UI (#61292)
Mis à jour par Frédéric Péters il y a environ 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
general: reduce logging infrastructure, do not expose anything in UI (#61292)