Development #10269
debug/webservice : avoir dans le journal les valeur de err/err_class/err_desc
0%
Description
Aujourd'hui on peut inscrire dans le journal les erreurs lors d'appels à des webservices, et une ligne d'info qui sera souvent le retour HTTP, genre "400 BAD REQUEST".
Juste pour nous aider au debug, qu'on n'ait pas à passer par les mails de trace et/ou un shell python pour charger le formdata, je propose d'inscrire les infos dans la page HTML, en commentaires.
<!-- err: 1 --> <!-- err_class: passerelle_paris_poc_gru.out_views.MissingKeyException --> <!-- err_desc: notification_sender is required -->
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 8 ans
- Fichier 0001-wscall-display-json-error-details-as-html-comments-i.patch 0001-wscall-display-json-error-details-as-html-comments-i.patch ajouté
- Statut changé de Nouveau à En cours
Mis à jour par Thomas Noël il y a environ 8 ans
Je préférerai que ça ne soit pas en commentaire mais directement visible.
Cas d'usage : un client qui nous signale le soucis, avec une copie d'écran, et où on n'a pas d'accès au backoffice.
Mis à jour par Frédéric Péters il y a environ 8 ans
Mais uniquement pour les personnes qui sont "un peu admin" quand même ?
Mis à jour par Thomas Noël il y a environ 8 ans
Non... Quand on affiche « 400 BAD REQUEST » je pense qu'on peut se permettre de donner d'autres détails aussi peu compréhensibles.
Mis à jour par Frédéric Péters il y a environ 8 ans
- Fichier 0001-wscall-display-json-error-details-in-journal-in-back.patch 0001-wscall-display-json-error-details-in-journal-in-back.patch ajouté
- Sujet changé de debug/webservice : avoir dans l'html du journal les valeur de err/err_class/err_desc à debug/webservice : avoir dans le journal les valeur de err/err_class/err_desc
- Patch proposed changé de Non à Oui
Voilà, patch modifié pour afficher les infos tirées de l'erreur, mettre ça derrière un titre dépliable, et étendre le message "Error during webservice call" avec le libellé de l'appel quand il y en a un.
Mis à jour par Thomas Noël il y a environ 8 ans
Zut, j'ai oublié de poser un Ack ici, j'ai pourtant relu (et testé) depuis longtemps
Mis à jour par Frédéric Péters il y a environ 8 ans
- Statut changé de En cours à Résolu (à déployer)
commit 4427805265bfe47843a8d494e8723a7d45784284 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Fri Mar 11 09:51:34 2016 +0100 wscall: display json error details in journal in backoffice (#10269)
wscall: display json error details in journal in backoffice (#10269)