Bug #14066
agoraplus: de l'unicode dans les message d'erreur remonte jusqu'à l'exception
Début:
24 novembre 2016
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
Dans le nouveau code de gestion des erreurs :
class AgoraAPIError(APIError): def __str__(self): return 'agora+ error: ' + super(AgoraAPIError, self).__str__()
mais on a parfois des raise AgoraAPIError(message_unicode_en_français_avec_des_accents)
qui fait crasher le str().
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a plus de 7 ans
- Fichier 0001-agoraplus-do-not-force-any-encoding-in-AgoraAPIError.patch 0001-agoraplus-do-not-force-any-encoding-in-AgoraAPIError.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Solution simple: remonter directement à ApiError, qui sera remontée jusqu'au décorateur JSON qui formatte les messages d'erreur des exception avec django.utils.translations.force_text, et tout va bien.
L'ajout de "agora+ error:" dans le message n'est pas nécessaire car on aura l'info err_class pour ça.
Mis à jour par Thomas Noël il y a plus de 7 ans
- Statut changé de En cours à Résolu (à déployer)
commit 3a5ad85a6e2d7add1d503e7c83eb352d3194c54d Author: Thomas NOEL <tnoel@entrouvert.com> Date: Thu Nov 24 08:35:36 2016 +0100 agoraplus: do not force any encoding in AgoraAPIError (#14066)
et posé sauvagement sur la recette.
Mis à jour par Benjamin Dauvergne il y a presque 6 ans
- Statut changé de Résolu (à déployer) à Fermé
agoraplus: do not force any encoding in AgoraAPIError (#14066)