Development #57253
ne pas afficher b' ' autour des bytes dans les logs
0%
Description
Dans les logs détaillés on va par exemple avoir :
body b'{"id_typeinterv": "5", "id_urgence": "1", "id_qualification": "3", "demandeur": " ", "observations": "Lampadaire \\u00e9teint \\u00e0 c\\u00f4t\\u00e9 du lyc\\u00e9e orsel au 30 rue orsel ", "elements": "OULDE032"}'
ce serait plus sympa d'avoir juste le message, pas le b' '.
Fichiers
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0001-misc-log-bytes-as-string-without-enclosing-b-57253.patch 0001-misc-log-bytes-as-string-without-enclosing-b-57253.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
En fait il n'y a pas de bytes dans les logs, car
def is_json_serializable(value): return isinstance(value, (list, dict, bool) + six.integer_types + six.string_types) attr['extra'] = {key: value for key, value in extra.items() if is_json_serializable(value)}
Si il y en a ils ont été manuellement convertis en chaîne avant le log, typiquement pour body on trouve extras.update({'body': repr(req.body[:max_size])})
.
Plein de manières de s'en sortir, je choisis d'autoriser d'envoyer des bytes dans extra
et de faire la conversion au moment du log en enlevant le b"".
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
Si payload ou content sont du texte, n'essayerait-on pas de le logger ça décodé en utf-8 plutôt que de logger un repr d'un bytes sans le "b" mais avec des caractères échapés ?
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
- Statut changé de Solution proposée à En cours
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0001-misc-log-bytes-as-string-without-enclosing-b-57253.patch 0001-misc-log-bytes-as-string-without-enclosing-b-57253.patch ajouté
- Statut changé de En cours à Solution proposée
Benjamin Dauvergne a écrit :
Si payload ou content sont du texte, n'essayerait-on pas de le logger ça décodé en utf-8
Oui je m'attendais à cette remarque, revoici
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Ok.
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 7109698192dc067c39a5b1031065a6af7056590d Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Tue Oct 5 17:06:02 2021 +0200 misc: log bytes as string without enclosing b"" (#57253)
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
misc: log bytes as string without enclosing b"" (#57253)