Projet

Général

Profil

Development #24375

enregistrer plus de contexte sur les erreurs de compute

Ajouté par Thomas Noël il y a presque 6 ans. Mis à jour il y a presque 6 ans.

Statut:
Nouveau
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
08 juin 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:

Description

Pour faire suite à #24327, avoir la possibilité d'envoyer un « log_context={'formdata': formdata, 'status_item': item} » dans l'appel à compute, et provoquerait un LoggerError dans compute en cas de crash.

L'objectif étant qu'il n'y ai plus besoin de faire un notify_of_exception sur un crash de compute.

Historique

#1

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

Et envoyer globals() et locals(), genre, aussi ? (mais là-dessus faudra réfléchir à nouveau au moment où il y aura une création paresseuse des variables)

#2

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

Frédéric Péters a écrit :

Et envoyer globals() et locals(), genre, aussi ? (mais là-dessus faudra réfléchir à nouveau au moment où il y aura une création paresseuse des variables)

L'idée est que compute dispose déjà d'un contexte (get_publisher().substitutions.get_context_variables()) et donc oui, l'envoyer à LoggerError.record en faisant en sorte que celui-ci l'enregistre et le restitue. Et oui, lorsqu'on sera devenus paresseux, faudra voir ; sans doute reprendre des choses de ce qu'on fera alors sur l'inspecteur d'une demande.

#3

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

  • Sujet changé de enregistrer les erreurs de compute à enregistrer plus de contexte sur les erreurs de compute

De fait, ça a été fait lors de #24645. Je change donc l'objet du ticket pour lui demander de voir ce qu'on pourrait faire pour avoir plus de contexte dans le LoggerError, c'est-à-dire le dictionnaire des variables envoyées au template ou à eval

Formats disponibles : Atom PDF