Development #56139
Appeler directement le log Django des réponses dans to_json pour en définir le niveau
0%
Description
Parce qu'on log déjà les 500 qu'on génère, il n'est pas nécessaire de le faire faire par Django en plus.
Fichiers
Demandes liées
Révisions associées
utils/jsonresponse: prevent Django from logging our 500 responses as errors (#56139)
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
- Fichier 0001-utils-jsonresponse-reorder-exception-cases-to-make-b.patch 0001-utils-jsonresponse-reorder-exception-cases-to-make-b.patch ajouté
- Fichier 0002-utils-jsonresponse-prevent-Django-from-logging-our-5.patch 0002-utils-jsonresponse-prevent-Django-from-logging-our-5.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
- Lié à Development #55516: logging des exceptions, avec 2.2 plus de traceback ajouté
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
Aussi on ne peut pas régler ça au niveau de settings.LOGGING (dans debian_config_common.py par exemple) pour le logger django.request parce qu'on dépend quand même de ce logger pour les traces lambda dans les autres briques ou même au niveau de passerelle en dehors de to_json.
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
- Fichier 0001-utils-jsonresponse-reorder-exception-cases-to-make-b.patch 0001-utils-jsonresponse-reorder-exception-cases-to-make-b.patch ajouté
- Fichier 0002-utils-jsonresponse-prevent-Django-from-logging-our-5.patch 0002-utils-jsonresponse-prevent-Django-from-logging-our-5.patch ajouté
Mauvais usage de log_response.
Mis à jour par Thomas Noël il y a plus de 2 ans
Benjamin Dauvergne a écrit :
Mauvais usage de log_response.
Still (le "response" est en trop au début des arguments, je pense)
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
- Fichier 0001-utils-jsonresponse-reorder-exception-cases-to-make-b.patch 0001-utils-jsonresponse-reorder-exception-cases-to-make-b.patch ajouté
- Fichier 0002-utils-jsonresponse-prevent-Django-from-logging-our-5.patch 0002-utils-jsonresponse-prevent-Django-from-logging-our-5.patch ajouté
Ben oui banane que je suis.
Mis à jour par Thomas Noël il y a plus de 2 ans
Vu le crash des tests jenkins, c'est pas encore ça... mais cette fois je ne sais pas quoi dire.
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
- Fichier 0001-utils-jsonresponse-reorder-exception-cases-to-make-b.patch 0001-utils-jsonresponse-reorder-exception-cases-to-make-b.patch ajouté
- Fichier 0002-utils-jsonresponse-prevent-Django-from-logging-our-5.patch 0002-utils-jsonresponse-prevent-Django-from-logging-our-5.patch ajouté
Tests corrigés.
Mis à jour par Thomas Noël il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
J'ai donc été voir le code de log_response pour comprendre que ça se termine par un response._has_been_logged = True qui fait qu'aucun autre log ne sera fait par Django. Et donc ici, on logue quand même les 500, mais en niveau "warning", et donc on évite que Django logue en niveau "error".
(malgré le commentaire ça ne m'a pas été si évident à comprendre sans lire le code de log-response)
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 4b574ff0c97e2b281f398c895069e1bf7c472463
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Fri Aug 13 10:17:09 2021 +0200
utils/jsonresponse: prevent Django from logging our 500 responses as errors (#56139)
commit 9eb682a1fb5491e11afeede35d276df52081a97d
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Fri Aug 13 10:16:05 2021 +0200
utils/jsonresponse: reorder exception cases to make behaviour clearer (#56139)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
utils/jsonresponse: reorder exception cases to make behaviour clearer (#56139)