Projet

Général

Profil

Bug #23613

crash sur /__skeleton__/ quand une page est une redirection avec une URL templatée qui n'existe pas

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
Thomas Noël
Version cible:
-
Début:
04 mai 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Une page est une redirection vers [porte_doc_url] et pas de fargo déployé, on va avoir ceci :

Internal Server Error: /__skeleton__/
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/django/core/handlers/base.py", line 132, in get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/usr/lib/python2.7/dist-packages/combo/public/views.py", line 195, in skeleton
    redirect_url = utils.get_templated_url(page.redirect_url)
  File "/usr/lib/python2.7/dist-packages/combo/utils/urls.py", line 62, in get_templated_url
    return re.sub(r'(\[.+?\])', repl, url)
  File "/usr/lib/python2.7/re.py", line 155, in sub
    return _compile(pattern, flags).sub(repl, string, count)
  File "/usr/lib/python2.7/dist-packages/combo/utils/urls.py", line 60, in repl
    raise TemplateError('unknown variable %s', varname)
TemplateError: unknown variable porte_doc_url

et les systèmes qui cherchent leur templates distant vont planter gravement (500 brutale et directe).

C'est lié à la recherche la page correspondante :

    for page in redirect_pages:
        redirect_url = utils.get_templated_url(page.redirect_url)  <-- crash TemplateError

On pourrait faire un « try ... except TemplateError: continue »

Fichiers

0001-skeleton-ignore-pages-with-bad-template-redirection-.patch (1,89 ko) 0001-skeleton-ignore-pages-with-bad-template-redirection-.patch Thomas Noël, 04 mai 2018 17:20

Révisions associées

Révision 8630b9ee (diff)
Ajouté par Thomas Noël il y a presque 6 ans

skeleton: ignore pages with bad template redirection (#23613)

Historique

#1

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

  • Assigné à mis à Thomas Noël
#2

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

#3

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

  • Statut changé de En cours à Résolu (à déployer)
commit db4436daff459c6ac523464d08c5905be7ee9bdb
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sat Apr 14 10:51:20 2018 +0200

    manager: do not crash on page configured with a missing template (#23186)
#4

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

Formats disponibles : Atom PDF