Bug #10999
Le formatage des logs des exceptions contenant de l'UTF-8 ne marche pas
Début:
19 mai 2016
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
Voir https://sentry.entrouvert.org/sentry/recette/issues/352/, en fait ces lignes de code là ne sont pas bonnes:
models.py: raise Exception((push.get('message') or '').encode('utf-8'))
models.py: raise Exception(push['message'].encode('utf-8'))
models.py: raise Exception(res['message'].encode('utf-8'))
models.py: raise Exception(res['message'].encode('utf-8'))
models.py: raise Exception(res['message'].encode('utf-8'))
models.py: raise Exception(res['erreur'].encode('utf-8'))
Il faut laisser les chaînes unicode dans l'exception car logging et django le gère très bien normalement (globalement il faut éviter encode() sauf si on sait très bien pourquoi l'utilise ou si on est dans w.c.s.).
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a presque 8 ans
- Fichier 0001-agoraplus-don-t-encode-exception-log-messages-10999.patch 0001-agoraplus-don-t-encode-exception-log-messages-10999.patch ajouté
- Statut changé de Nouveau à En cours
- Assigné à mis à Thomas Noël
- Patch proposed changé de Non à Oui
Je ne sais plus ce qui m'avait poussé à cette bêtise.
Mis à jour par Thomas Noël il y a presque 8 ans
- Statut changé de En cours à Résolu (à déployer)
commit 0a245be1cea20a1e9e5f833d0e482e5008b86bef
Author: Thomas NOEL <tnoel@entrouvert.com>
Date: Wed Jun 8 17:06:07 2016 +0200
agoraplus: don't encode exception log messages (#10999)
Mis à jour par Benjamin Dauvergne il y a presque 6 ans
- Statut changé de Résolu (à déployer) à Fermé
agoraplus: don't encode exception log messages (#10999)