Projet

Général

Profil

Development #54388

contrôler la syntaxe posée en gabarit (TemplateSyntaxError: Invalid filter: 'datetime')

Ajouté par Sentry Io il y a presque 3 ans. Mis à jour il y a presque 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
28 mai 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

https://sentry.entrouvert.org/entrouvert/publik/issues/47265/

TemplateSyntaxError: Invalid filter: 'datetime'
(14 additional frame(s) were not displayed)
...
  File "<unknown source>", line 1
    {{ event.start_datetime|datetime:&quot;l&quot; }} {{event.start_datetime}}

Un gabarit invalide a été configuré et ça casse derrière; on devrait valider que ça passe et/ou de toute façon ne pas planter dans l'api datetimes.


Fichiers

Révisions associées

Révision 2921ed52 (diff)
Ajouté par Valentin Deniaud il y a presque 3 ans

api: prevent crash on invalid event_display_template (#54388)

Historique

#1

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

  • Projet changé de Suivi des traces à Chrono
#2

Mis à jour par Valentin Deniaud il y a presque 3 ans

  • Assigné à mis à Valentin Deniaud
#3

Mis à jour par Valentin Deniaud il y a presque 3 ans

#4

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

Le template_validator il va laisser passer le |datetime et autres erreurs du genre, non ? (parce que la syntaxe en soit est valide)

#5

Mis à jour par Valentin Deniaud il y a presque 3 ans

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

Le template_validator il va laisser passer le |datetime et autres erreurs du genre, non ? (parce que la syntaxe en soit est valide)

Ouep ça ne protège même pas de grand chose, {{ a a }} passe pas mais {{ a passe. Compliqué de faire mieux cependant vu qu'on a pas le contexte d'évaluation à ce moment là.

#6

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

Créer un objet event bidon et le passer dans le contexte et faire un render et voir si ça saute ?

#8

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

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

Mis à jour par Valentin Deniaud il y a presque 3 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 2921ed52a33562ca0e801ebb5f20dcdd16269e55
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Mon May 31 17:11:27 2021 +0200

    api: prevent crash on invalid event_display_template (#54388)
#10

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

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

Formats disponibles : Atom PDF