Project

General

Profile

Development #37491

permettre d'ajouter du contexte au proxylogger

Added by Benjamin Dauvergne 4 months ago. Updated 3 months ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
06 Nov 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

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.

0001-base-add-context-method-to-ProxyLogger-37491.patch View (1.56 KB) Benjamin Dauvergne, 15 Nov 2019 04:54 PM

0001-base-add-context-method-to-ProxyLogger-37491.patch View (1.56 KB) Benjamin Dauvergne, 15 Nov 2019 05:19 PM

0001-base-add-context-method-to-ProxyLogger-37491.patch View (1.56 KB) Benjamin Dauvergne, 16 Nov 2019 05:30 PM

0001-base-add-context-method-to-ProxyLogger-37491.patch View (1.56 KB) Benjamin Dauvergne, 16 Nov 2019 05:31 PM

0001-base-add-context-method-to-ProxyLogger-37491.patch View (3.08 KB) Benjamin Dauvergne, 28 Nov 2019 03:17 PM


Related issues

Related to Passerelle - Development #35818: mdel: démarche DDPACS Solution déployée 04 Sep 2019

Associated revisions

Revision 8316231a (diff)
Added by Benjamin Dauvergne 3 months ago

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.

History

#1 Updated by Benjamin Dauvergne 3 months ago

#2 Updated by Benjamin Dauvergne 3 months ago

#3 Updated by Frédéric Péters 3 months ago

(un seul n à dictionary) (ceci n'est pas une relecture)

#5 Updated by Benjamin Dauvergne 3 months ago

Sans rebase sur #37482 ça marche moins bien.

#7 Updated by Emmanuel Cazenave 3 months ago

Tu pourrais pointer le cas d'usage ?

Parce que là je comprends pas grand chose au truc.

#8 Updated by Benjamin Dauvergne 3 months ago

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}).

#9 Updated by Emmanuel Cazenave 3 months ago

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).

#10 Updated by Benjamin Dauvergne 3 months ago

T'as raison c'est beaucoup mieux, la fatigue certainement. Je t'ai mis un test aussi.

#11 Updated by Emmanuel Cazenave 3 months ago

  • Status changed from Solution proposée to Solution validée

#12 Updated by Benjamin Dauvergne 3 months ago

  • Status changed from Solution validée to 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.

#13 Updated by Frédéric Péters 3 months ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF