Add context to logger

[silverpark]

Bonjour,

J'ai rajouté la possibilité d'ajouter un context au logger, pour ce faire j'ai rajouté à la fonction doLog de sfLoggerInterface le paramètre $context qui lui est utilisé par tous les logger lors de l'implémentation de cette interface.

Je propose cette modification pour par exemple envoyer un context (variable etc..) vers sentry (par le biais de raven).

Exemple :

$this->getLogger()->err($e, array('variableTestToShow' => 'variableTestShow'));

J'ai ajouté cette fonctionnalité sur mes projets mais j'aimerais tout de même vous la proposer, c'est comme ça que symfony2 procède dans le Logger.php

Qu'en pensez-vous ?

Cordialement.

[j0k3r]

Il faudrait modifier les tests dans sfLoggerTest et sfLoggerWrapperTest.

[silverpark]

Très bien je m'en occupe.

[GromNaN]

Ca casse la rétro-compatibilité si des personnes ont développé leur propre logger car l'interface et la signature des méthodes est modifiée. Ce n'est pas notre cas, mais ça peut bloquer des personnes qui ont déjà migré leur projet sur cette version de symfony1.

[silverpark]

Je suis d'accord avec vous. C'est pour cela que j'hésitais à vous le proposer. Si vous passez au niveau des tags en version supérieur cela pose t-il problème ?

J'ai également fait une mise à jour au niveau du sfRavenPlugin pour qu'il envoie vers sentry le context.