Project

General

Profile

Development #57253

ne pas afficher b' ' autour des bytes dans les logs

Added by Frédéric Péters 23 days ago. Updated 9 days ago.

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

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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' '.


Files

Associated revisions

Revision 71096981 (diff)
Added by Valentin Deniaud 11 days ago

misc: log bytes as string without enclosing b"" (#57253)

History

#1

Updated by Valentin Deniaud 12 days ago

  • Assignee set to Valentin Deniaud
#2

Updated by Valentin Deniaud 12 days ago

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"".

#3

Updated by Benjamin Dauvergne 12 days ago

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 ?

#4

Updated by Benjamin Dauvergne 12 days ago

  • Status changed from Solution proposée to En cours
#5

Updated by Valentin Deniaud 11 days ago

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

#6

Updated by Benjamin Dauvergne 11 days ago

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

Ok.

#7

Updated by Valentin Deniaud 11 days ago

  • Status changed from Solution validée to 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)
#8

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

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

Also available in: Atom PDF