Project

General

Profile

Development #55437

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

Added by Frédéric Péters 3 months ago. Updated 13 days ago.

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

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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)

Files

Associated revisions

Revision e27f5b7e (diff)
Added by Paul Marillonnet 16 days ago

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

History

#1

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

  • Description updated (diff)
#2

Updated by Paul Marillonnet 3 months ago

  • Assignee set to Paul Marillonnet
  • Status changed from Nouveau to En cours
#3

Updated by Paul Marillonnet 3 months ago

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

#4

Updated by Paul Marillonnet 3 months ago

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

Updated by Benjamin Dauvergne about 2 months ago

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

#7

Updated by Paul Marillonnet about 2 months ago

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

Updated by Benjamin Dauvergne about 1 month ago

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

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

#10

Updated by Lauréline Guerin 16 days ago

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

#11

Updated by Benjamin Dauvergne 16 days ago

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

Updated by Paul Marillonnet 15 days ago

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

Updated by Frédéric Péters 13 days ago

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

Also available in: Atom PDF