Projet

Général

Profil

Development #56802

supprimer le mode d'affichage des exceptions "affichage en HTML"

Ajouté par Frédéric Péters il y a plus de 2 ans. Mis à jour il y a plus de 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
08 septembre 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Ça foire au fond de quixote,

  File "wcs/qommon/publisher.py", line 318, in finish_failed_request
    error_page = self._generate_cgitb_error(request, original_response, exc_type, exc_value, tb)
  File "quixote/publish.py", line 212, in _generate_cgitb_error
    s = str(util.dump_request(request))
  File "quixote/util.py", line 342, in dump_request
    r += row_fmt % (k, v)

https://sentry.entrouvert.org/entrouvert/publik/issues/51971/ & https://sentry.entrouvert.org/entrouvert/publik/issues/51972/ & https://sentry.entrouvert.org/entrouvert/publik/issues/51973/

(c'est sur un site qui a été configuré pour envoyer les traces pas à nous, du coup il y a juste cette trace courte dans sentry sans possibilité de savoi ce qu'il pouvait y avoir dans k et v).

C'est une option rarement utilisée et pour moi on pourrait juste la retirer.


Fichiers

Révisions associées

Révision 67f35f3b (diff)
Ajouté par Frédéric Péters il y a plus de 2 ans

settings: remove option to set exception display style (#56802)

Historique

#2

Mis à jour par Frédéric Péters il y a plus de 2 ans

  • Statut changé de Nouveau à En cours
  • Assigné à mis à Frédéric Péters

(à la base j'étais même pour virer totalement l'option et afficher en texte quand la case "mode debug" est cochée, si c'est considéré raisonnable je peux aussi faire ce patch).

#3

Mis à jour par Thomas Noël il y a plus de 2 ans

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

(à la base j'étais même pour virer totalement l'option et afficher en texte quand la case "mode debug" est cochée, si c'est considéré raisonnable je peux aussi faire ce patch).

Pour moi oui, c'est raisonnable ainsi.

#4

Mis à jour par Frédéric Péters il y a plus de 2 ans

#5

Mis à jour par Lauréline Guérin il y a plus de 2 ans

  • Statut changé de Solution proposée à Solution validée
#6

Mis à jour par Frédéric Péters il y a plus de 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 67f35f3be4ef927a065163cc766754822d85c270
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri Sep 10 15:45:22 2021 +0200

    settings: remove option to set exception display style (#56802)
#7

Mis à jour par Frédéric Péters il y a plus de 2 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF