Projet

Général

Profil

Bug #47014

crash sur commentaire avec un {{ webservice.foo }} dont l'URL est vide

Ajouté par Thomas Noël il y a plus de 3 ans. Mis à jour il y a 5 mois.

Statut:
Fermé
Priorité:
Bas
Assigné à:
-
Version cible:
-
Début:
25 septembre 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Un commentaire {{ webservice.foo }}, webservice qui appelle une URL du genre {{ bar }}, donc vide au calcul.

wcs crashe lorsqu'il cherche à afficher ce formulaire.

Historique

#1

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

à noter qu'en backoffice pas de plantage grâce à ce try/except dans wcs/admin/forms.py :

    def get_preview(self):
...
            if hasattr(field, str('add_to_form')):
                try:
                    field.add_to_form(form)
                except Exception as e:
                    form.widgets.append(HtmlWidget(htmltext(
                        '<div class="errornotice"><p>%s (%s)</p></div>') % (
                            _('Error previewing field.'),
                            e)))
#2

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

  • Priorité changé de Normal à Bas

En fait il faut plutôt faire que l'appel à webservice.foo ne crashe pas wcs.

Je descend la priorité de ce ticket, je ne vois pas quoi faire d'autre qu'une 500 quand un champ crashe (ne pas l'afficher serait risqué).

Ticket à rejeter si l'erreur 500 reste audible.

#3

Mis à jour par Frédéric Péters il y a 5 mois

  • Statut changé de Nouveau à Fermé

Ticket à rejeter si l'erreur 500 reste audible.

Depuis il n'y a plus d'erreur 500, l'erreur se trouve enregistrée et le champ commentaire est présenté vide. (sans la trace de l'époque c'est compliqué de tracer au ticket qui a corrigé ça).

Formats disponibles : Atom PDF