Projet

Général

Profil

Development #55437

remplacer les get_logger().error() par des appels à record_error

Ajouté par Frédéric Péters il y a presque 3 ans. Mis à jour il y a plus de 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Paul Marillonnet
Version cible:
-
Début:
06 juillet 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

get_logger().error() ça va juste, quand c'est activé, aller écrire dans un fichier log local jamais lu (et qui a des problèmes comme #36176).

On devrait remplacer tout ça par des appels à record_error().

Ou dans certaines occasions simplement supprimer le logging, je vois par exemple

wcs/qommon/saml2.py:                get_logger().error('%s unavailable - lasso is not installed' % rel_path)

Fichiers

0001-misc-replace-get_logger-.error-with-newer-record_err.patch (28,2 ko) 0001-misc-replace-get_logger-.error-with-newer-record_err.patch Paul Marillonnet, 12 juillet 2021 10:20
0001-misc-replace-get_logger-.error-with-newer-record_err.patch (28,1 ko) 0001-misc-replace-get_logger-.error-with-newer-record_err.patch Paul Marillonnet, 13 juillet 2021 11:35
0001-misc-replace-get_logger-.error-with-newer-record_err.patch (26,4 ko) 0001-misc-replace-get_logger-.error-with-newer-record_err.patch Paul Marillonnet, 06 août 2021 09:38
0001-misc-replace-get_logger-.error-with-newer-record_err.patch (39,7 ko) 0001-misc-replace-get_logger-.error-with-newer-record_err.patch Benjamin Dauvergne, 11 septembre 2021 18:16

Révisions associées

Révision e27f5b7e (diff)
Ajouté par Paul Marillonnet il y a plus de 2 ans

misc: replace get_logger().error() with newer record_error() (#55437)

Historique

#1

Mis à jour par Frédéric Péters il y a presque 3 ans

  • Description mis à jour (diff)
#2

Mis à jour par Paul Marillonnet il y a presque 3 ans

  • Statut changé de Nouveau à En cours
  • Assigné à mis à Paul Marillonnet
#3

Mis à jour par Paul Marillonnet il y a presque 3 ans

Note pour moi-même : à passer après #55414, ne pas se marcher sur les pieds etc.

#4

Mis à jour par Paul Marillonnet il y a presque 3 ans

Paul Marillonnet a écrit :

Note pour moi-même : à passer après #55414, ne pas se marcher sur les pieds etc.

Et en fait non, on ne se marche pas sur les pieds.

#6

Mis à jour par Benjamin Dauvergne il y a plus de 2 ans

Je ne comprends pas les changements à test_datasource.py, je ne vois aucun changement dans les sources qui corresponde.

#7

Mis à jour par Paul Marillonnet il y a plus de 2 ans

Benjamin Dauvergne a écrit :

Je ne comprends pas les changements à test_datasource.py, je ne vois aucun changement dans les sources qui corresponde.

Ah oui, des mystères de l’exécution locale des tests w.c.s. que je n’ai pas encore résolus. Changements superflus que j’ai retirés dans ce présent patch.

#8

Mis à jour par Benjamin Dauvergne il y a plus de 2 ans

  • Statut changé de Solution proposée à Solution validée

J'ai rebasé la branche, à pousser si c'est toujours vert.

#9

Mis à jour par Benjamin Dauvergne il y a plus de 2 ans

Rebasé.

#10

Mis à jour par Lauréline Guérin il y a plus de 2 ans

pourquoi tout ce code en plus dans wcs/backoffice/management.py ?

#11

Mis à jour par Benjamin Dauvergne il y a plus de 2 ans

Lauréline Guerin a écrit :

pourquoi tout ce code en plus dans wcs/backoffice/management.py ?

J'ai rebasé comme un cochon, heureusement pylint m'a montré le problème.

#12

Mis à jour par Paul Marillonnet il y a plus de 2 ans

  • Statut changé de Solution proposée à Résolu (à déployer)
commit e27f5b7e6617cb3aeef907a855212380f35f0651
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date:   Thu Jul 8 11:46:00 2021 +0200

    misc: replace get_logger().error() with newer record_error() (#55437)
#13

Mis à jour par Frédéric Péters il y a plus de 2 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF