Projet

Général

Profil

Bug #22380

Pas d'interpolation dans les messages de ProxyLogger

Ajouté par Emmanuel Cazenave il y a environ 6 ans. Mis à jour il y a presque 6 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
08 mars 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Dans passerlle.base.models.ProxyLogger._log on a
attr['message'] = message[:ResourceLog._meta.get_field('message').max_length] qui ignore royalement les interpolations
demandées par l'appelant à travers *args.


Fichiers

Révisions associées

Révision 59283d33 (diff)
Ajouté par Emmanuel Cazenave il y a environ 6 ans

handle string interpolation in ProxyLogger (#22380)

Historique

#1

Mis à jour par Emmanuel Cazenave il y a environ 6 ans

J'en profite pour :

  • accepter un dict pour les interpolations (comme dans stdlib à laquelle je pique 2 lignes de code)
  • faire le boulot préalable à la création d'un ResourceLog uniquement si .... on crée un ResourceLog
#2

Mis à jour par Josué Kouka il y a environ 6 ans

Ok pour moi, par contre:
- Typo

+            # RIPOFF from pyton stdlib logging/__init__.py

- Vu qu'un nouveau fichier(tests/test_proxylogger.py) pour les tests a été ajouté, peut etre déplacer tests/test_generic_endpoint.py::test_proxy_logger dans ce nouveau fichier.

#3

Mis à jour par Emmanuel Cazenave il y a environ 6 ans

Corrigé la typo, et message de commit qui commence maintenant par un verbe.
Je laisse les tests là où ils sont parce ceux de test_generic_endpoint.py sont de plus haut niveau.

#4

Mis à jour par Thomas Noël il y a environ 6 ans

Ack (mais à pousser lundi ou mardi)

#5

Mis à jour par Emmanuel Cazenave il y a environ 6 ans

  • Statut changé de En cours à Résolu (à déployer)
commit 59283d3386fb88566428cb345cb4a10e74a47c7a
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date:   Thu Mar 8 16:07:28 2018 +0100

    handle string interpolation in ProxyLogger (#22380)

#6

Mis à jour par Emmanuel Cazenave il y a environ 6 ans

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

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

  • Statut changé de Solution déployée à Fermé

Formats disponibles : Atom PDF