Bug #22380
Pas d'interpolation dans les messages de ProxyLogger
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
Historique
Mis à jour par Emmanuel Cazenave il y a environ 6 ans
- Fichier 0001-string-interpolation-in-ProxyLogger-22380.patch 0001-string-interpolation-in-ProxyLogger-22380.patch ajouté
- Statut changé de Nouveau à En cours
- Assigné à mis à Emmanuel Cazenave
- Patch proposed changé de Non à Oui
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 unResourceLog
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.
Mis à jour par Emmanuel Cazenave il y a environ 6 ans
- Fichier 0001-handle-string-interpolation-in-ProxyLogger-22380.patch 0001-handle-string-interpolation-in-ProxyLogger-22380.patch ajouté
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.
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)
Mis à jour par Emmanuel Cazenave il y a environ 6 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Benjamin Dauvergne il y a presque 6 ans
- Statut changé de Solution déployée à Fermé
handle string interpolation in ProxyLogger (#22380)