Development #12566
journal des erreurs
0%
Description
Ajouter un objet genre LoggedError qui aurait le contenu de la trace et une référence à l'origine de celle-ci (la plus précise possible, pour par exemple pouvoir remonter directement à l'action d'appel webservice d'où vient l'erreur).
La liste intégrale serait accessible mais surtout il y aurait un pavé avec les erreurs récentes sur la page d'atelier d'un formulaire/workflow.
Fichiers
Demandes liées
Révisions associées
backoffice: add pages to display formdef/workflow errors (#12566)
Historique
Mis à jour par Frédéric Péters il y a presque 7 ans
- Lié à Development #6102: Notification des exceptions liées à un formulaire/workflow ajouté
Mis à jour par Frédéric Péters il y a presque 7 ans
- Fichier 0002-backoffice-add-pages-to-display-formdef-workflow-err.patch 0002-backoffice-add-pages-to-display-formdef-workflow-err.patch ajouté
- Fichier 0001-general-record-formdef-workflow-exceptions-12566.patch 0001-general-record-formdef-workflow-exceptions-12566.patch ajouté
- Fichier Capture d’écran 2017-05-25 à 14.29.47.png Capture d’écran 2017-05-25 à 14.29.47.png ajouté
- Fichier Capture d’écran 2017-05-25 à 14.29.36.png Capture d’écran 2017-05-25 à 14.29.36.png ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
0001 pour ajouter la technique, 0002 pour ajouter l'UI. Captures en bonus.
Mis à jour par Pierre Cros il y a presque 7 ans
On rajouterai pas dans le message affichée au user le fait que l'erreur a été envoyée à admin@entrouvert.com ? Ou alors on souhaite qu'il nous renvoie cette erreur potentiellement ?
Mis à jour par Frédéric Péters il y a presque 7 ans
- Fichier 0002-backoffice-add-pages-to-display-formdef-workflow-err.patch 0002-backoffice-add-pages-to-display-formdef-workflow-err.patch ajouté
- Fichier Capture d’écran 2017-05-26 à 11.31.02.png Capture d’écran 2017-05-26 à 11.31.02.png ajouté
Patch d'UI modifié pour couper la trace sur les titres.
On rajouterai pas dans le message affichée au user le fait que l'erreur a été envoyée à admin@entrouvert.com ?
Je viens d'ajouter mais ce n'est pas nécessairement une vraie info, si le paramétrage de debug a été changé entre temps, on n'affichera pas quelque chose de correct. Plus tard on pourra brancher pour de vrai les notifs à ce système, ça deviendra un mini-sentry, ça nous évitera cent messages pour la même trace, etc. Mais pour le moment, si on tape l'info, c'est un peu pipeau.
Ou alors on souhaite qu'il nous renvoie cette erreur potentiellement ?
Qu'ils nous tapent l'url dans un ticket, je dirais.
Mis à jour par Thomas Noël il y a presque 7 ans
Frédéric Péters a écrit :
Patch d'UI modifié pour couper la trace sur les titres.
Je viens d'ajouter mais ce n'est pas nécessairement une vraie info, si le paramétrage de debug a été changé entre temps, on n'affichera pas quelque chose de correct.
Peut-être, pour mentir moins, ajouter un attribut error_email à LoggedError, qui sera positionné à publisher.logger.error_email au moment du error.occurences_count += 1 ?
Mis à jour par Thomas Noël il y a presque 7 ans
Relu une nouvelle fois : allons-y dès maintenant pour poser cela rapidement sur la recette.
Ack.
Mis à jour par Frédéric Péters il y a presque 7 ans
- Statut changé de En cours à Résolu (à déployer)
commit b056db3e32a80a277ad68cde9f72f4affbd06511 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Thu May 25 14:26:46 2017 +0200 backoffice: add pages to display formdef/workflow errors (#12566) commit 3ce047903e453f69eb647c3b59547aa4465fe920 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Thu May 25 12:22:20 2017 +0200 general: record formdef/workflow exceptions (#12566)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
general: record formdef/workflow exceptions (#12566)