Projet

Général

Profil

Bug #11140

dans le décorateur json, logguer la requête lors d'exceptions

Ajouté par Frédéric Péters il y a presque 8 ans. Mis à jour il y a presque 6 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
30 mai 2016
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Error occurred while processing request
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/passerelle/utils/jsonresponse.py", line 349, in api
    resp = f(*args, **kwargs)
  File "/usr/lib/python2.7/dist-packages/passerelle/contrib/iparapheur/views.py", line 76, in get
    return self.get_data(request, *args, **kwargs)
  File "/usr/lib/python2.7/dist-packages/passerelle/contrib/iparapheur/views.py", line 138, in get_data
    return self.get_object().get_file_status(kwargs['file_id'])
  File "/usr/lib/python2.7/dist-packages/passerelle/contrib/iparapheur/models.py", line 123, in get_file_status
    raise FileError(resp.MessageRetour.message)
FileError: Le dossierID '37fb2d75-36fa-4bd3-8701-0cd533ecafba' est inconnu dans le Parapheur.

Request repr(): unavailable

C'est parce que :

> /usr/lib/python2.7/dist-packages/django/utils/log.py(105)emit()
-> request = record.request
(Pdb) n
AttributeError: "'LogRecord' object has no attribute 'request'" 

Fichiers


Demandes liées

Lié à Passerelle - Development #11267: jsonresponse: passer la requête dans le dico extras quand une exception est journaliséeRejeté08 juin 2016

Actions

Révisions associées

Révision b08977d9 (diff)
Ajouté par Frédéric Péters il y a presque 8 ans

jsonresponse: pass request object to logging (#11140)

Historique

#1

Mis à jour par Frédéric Péters il y a presque 8 ans

Ça peut être plus compliqué que ça, je passe peut-être à côté de trucs, mais sur ce cas précis, ce patch me semble fonctionner.

#2

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

Ack.

#3

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 8 ans

Autant de virer req.method, non?

#4

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

C'est pour graylog ça je pense.

#5

Mis à jour par Frédéric Péters il y a presque 8 ans

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

C'est pour graylog ça je pense.

C'est aussi ce que j'ai pensé.

commit b08977d9646f214d04268969d455f3cb60ff2796
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Mon May 30 23:29:35 2016 +0200

    jsonresponse: pass request object to logging (#11140)
#6

Mis à jour par Thomas Noël il y a presque 8 ans

  • Lié à Development #11267: jsonresponse: passer la requête dans le dico extras quand une exception est journalisée ajouté
#7

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

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

Formats disponibles : Atom PDF