Bug #30558
log des payloads
Début:
12 février 2019
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
Suite de #30540, il y a jsonresponse.py qui contient,
if req.method == 'POST':
extras.update({'body': req.body})
et donc qui craque de la même manière sur un payload pas utf-8.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 5 ans
- Fichier 0001-misc-don-t-log-payload-as-is-use-repr-so-it-can-be-s.patch 0001-misc-don-t-log-payload-as-is-use-repr-so-it-can-be-s.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Et là je me dis qu'ailleurs on coupe le payload à 5000 caractères, qu'on devrait appliquer la même affaire partout (en imaginant aussi, plus tard, faire de ça un paramètre de configuration des logs).
Voilà patch qui repr() et coupe à 5000.
(branche wip poussée, aussi).
Mis à jour par Thomas Noël il y a environ 5 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a environ 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 3f131170216c47e02c6b5d5fa5a33cd733cbb72f
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Tue Feb 12 12:05:40 2019 +0100
misc: don't log payload as is, use repr() so it can be stored (#30558)
Mis à jour par Frédéric Péters il y a environ 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Statut changé de Solution déployée à Fermé
misc: don't log payload as is, use repr() so it can be stored (#30558)