Projet

Général

Profil

Development #10283

lever l'exception même quand le décorateur to_json la "cache" sous forme d'une erreur http

Ajouté par Serghei Mihai il y a environ 8 ans. Mis à jour il y a environ 7 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Serghei Mihai
Version cible:
-
Début:
13 mars 2016
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

On doit pouvoir recevoir les traces par mail, sentry.
Et au passage inclure django-jsonresponse(https://github.com/jjay/django-jsonresponse) dans passerelle afin d'éviter de packager une dependence de plus.

Fichiers

0001-embed-jsonresponse-into-the-package-10283.patch (28 ko) 0001-embed-jsonresponse-into-the-package-10283.patch Serghei Mihai, 30 mars 2016 19:10
0001-embed-jsonresponse-into-the-package-10283.patch (28,7 ko) 0001-embed-jsonresponse-into-the-package-10283.patch Serghei Mihai, 30 mars 2016 23:50
0001-embed-jsonresponse-into-the-package-10283.patch (28 ko) 0001-embed-jsonresponse-into-the-package-10283.patch Serghei Mihai, 31 mars 2016 09:43
0001-embed-jsonresponse-into-the-package-10283.patch (32,4 ko) 0001-embed-jsonresponse-into-the-package-10283.patch Serghei Mihai, 13 avril 2016 16:41

Révisions associées

Révision 9ba00c70 (diff)
Ajouté par Serghei Mihai il y a environ 8 ans

embed jsonresponse into the package (#10283)

Historique

#1

Mis à jour par Serghei Mihai il y a environ 8 ans

  • Fichier 0001-embed-jsonresponse-into-the-package-10283.patch ajouté
  • Statut changé de Nouveau à En cours
  • Patch proposed changé de Non à Oui

Je propose d'inclure jsonresponse dans passerelle.utils et pour recevoir les erreurs par mail, sentry déclarer dans debian_config_common de hobo:

'loggers': {
        ...
        'jsonresponse': {
            'level': 'ERROR',
        },
        ...
}

#2

Mis à jour par Serghei Mihai il y a environ 8 ans

  • Fichier 0001-embed-jsonresponse-into-the-package-10283.patch supprimé
#3

Mis à jour par Serghei Mihai il y a environ 8 ans

Patch à jour pour définir le logger dans le debian_config.py du paquet.

#4

Mis à jour par Benjamin Dauvergne il y a environ 8 ans

Je préférerai qu'on log dans passerelle ou passerelle.jsonresponse ou passerelle.utils mais pas juste jsonresponse. Il faudrait modifier le README pour signaler qu'on a intégré jsonresponse (voir modèle dans le README de w.c.s.)

Comme pour les loggers requests il faudrait logger dans extra method, et body si POST. Le chemin est normalement déjà dans les logs grâce au LogFilter de hobo. Je ne sais pas si c'est le bon endroit mais j'en profiterai éventuellement pour logger en mode DEBUG toutes les requêtes et les réponses.

#5

Mis à jour par Serghei Mihai il y a environ 8 ans

  • Fichier 0001-embed-jsonresponse-into-the-package-10283.patch ajouté

Ok.
Pour logguer les requetes et les réponses je fais un ticket séparé.

#6

Mis à jour par Serghei Mihai il y a environ 8 ans

  • Fichier 0001-embed-jsonresponse-into-the-package-10283.patch supprimé
#8

Mis à jour par Benjamin Dauvergne il y a environ 8 ans

Je virerai le logger passerelle.jsonresponse des settings, la configuration sur le logger racine dans debian_config_common.py suffit (niveau INFO).

#10

Mis à jour par Benjamin Dauvergne il y a environ 8 ans

C'est bon tu peux écrire les tests :) (pytest-capturelog pour vérifier les logs)

#12

Mis à jour par Josué Kouka il y a environ 8 ans

Ack

#13

Mis à jour par Serghei Mihai il y a environ 8 ans

  • Statut changé de En cours à Résolu (à déployer)
commit 9ba00c7024d599944939b3a27a58586b50666615
Author: Serghei Mihai <smihai@entrouvert.com>
Date:   Mon Mar 14 12:32:53 2016 +0100

    embed jsonresponse into the package (#10283)
#14

Mis à jour par Serghei Mihai il y a environ 7 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF