Development #37491
permettre d'ajouter du contexte au proxylogger
0%
Description
Dans certaines situations on peut vouloir ajouter du contexte (dans le kwarg "extra") une seule fois pour tous les appels de log d'un objet.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Lié à Development #35818: mdel: démarche DDPACS ajouté
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Fichier 0001-base-add-context-method-to-ProxyLogger-37491.patch 0001-base-add-context-method-to-ProxyLogger-37491.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus de 4 ans
(un seul n à dictionary) (ceci n'est pas une relecture)
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Fichier 0001-base-add-context-method-to-ProxyLogger-37491.patch 0001-base-add-context-method-to-ProxyLogger-37491.patch ajouté
un seul n à dictionary.
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Fichier 0001-base-add-context-method-to-ProxyLogger-37491.patch 0001-base-add-context-method-to-ProxyLogger-37491.patch ajouté
Sans rebase sur #37482 ça marche moins bien.
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Fichier 0001-base-add-context-method-to-ProxyLogger-37491.patch 0001-base-add-context-method-to-ProxyLogger-37491.patch ajouté
Mauvais ticket.
Mis à jour par Emmanuel Cazenave il y a plus de 4 ans
Tu pourrais pointer le cas d'usage ?
Parce que là je comprends pas grand chose au truc.
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
Emmanuel Cazenave a écrit :
Tu pourrais pointer le cas d'usage ?
Parce que là je comprends pas grand chose au truc.
Ça simplifie du code de log dans le connecteur DDPACS, #35818 ça permet de logger de manière plus structurée (j'entre dans un endpoint, je définis mon context, genre le numéro de dossier, de la famille, whatever, et je logge tranquillement en debug, info ou erreur sans avoir à répéter extra={'filenumber': filenumber, 'familynumber': yyy}).
Mis à jour par Emmanuel Cazenave il y a plus de 4 ans
Je suis un peu dubatatif sur l'implémentation de context , pourquoi pas un simple return ProxyLogger(self.connector, extra) avec une adaptation de ProxyLogger.__init__ pour accepter extra ? (je trouverais ça straightforward).
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Fichier 0001-base-add-context-method-to-ProxyLogger-37491.patch 0001-base-add-context-method-to-ProxyLogger-37491.patch ajouté
T'as raison c'est beaucoup mieux, la fatigue certainement. Je t'ai mis un test aussi.
Mis à jour par Emmanuel Cazenave il y a plus de 4 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 8316231a22822ecd75887fb08158f8af4f853661
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Thu Oct 24 16:16:21 2019 +0200
base: add context method to ProxyLogger (#37491)
It copies the logger and add contextual informations to the extra dictionary,
it overwrites keys from previous calls to context.Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
base: add context method to ProxyLogger (#37491)
It copies the logger and add contextual informations to the extra dictionary,
it overwrites keys from previous calls to context.