Development #67381
opengis, validation du gabarit donné pour l'indexation pour les requêtes
0%
Description
Il n'y a actuellement pas de validation résultat on peut avoir des gabarits invalides et une TemplateSyntaxError remontée lors de l'indexation.
ex: https://sentry.entrouvert.org/entrouvert/publik/issues/62501/
Fichiers
Révisions associées
Historique
Mis à jour par A. Berriot il y a presque 2 ans
- Fichier 0001-opengis-validate-indexing-template-to-avoid-later-cr.patch 0001-opengis-validate-indexing-template-to-avoid-later-cr.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par A. Berriot il y a presque 2 ans
Mis à jour par A. Berriot il y a presque 2 ans
Le build fail mais je ne comprends pas pourquoi: https://jenkins.entrouvert.org/job/passerelle-wip/job/wip%252F67381-opengis-validation-du-gabarit-do/2/console
Mis à jour par A. Berriot il y a presque 2 ans
Mis à jour par A. Berriot il y a presque 2 ans
Agate Berriot a écrit :
Le build fail mais je ne comprends pas pourquoi: https://jenkins.entrouvert.org/job/passerelle-wip/job/wip%252F67381-opengis-validation-du-gabarit-do/2/console
trouvé, c'était isort
Mis à jour par Valentin Deniaud il y a presque 2 ans
Peut-être plus simple d'utiliser le validate_template dans passerelle.utils.templates ?
Mis à jour par A. Berriot il y a presque 2 ans
Mis à jour par A. Berriot il y a presque 2 ans
Valentin Deniaud a écrit :
Peut-être plus simple d'utiliser le validate_template dans passerelle.utils.templates ?
bien vu, merci !
Mis à jour par Valentin Deniaud il y a presque 2 ans
Possible d'ajouter ça plutôt au niveau du modèle, genre validators=[validate_template]
?
Mis à jour par A. Berriot il y a presque 2 ans
Mis à jour par A. Berriot il y a presque 2 ans
Valentin Deniaud a écrit :
Possible d'ajouter ça plutôt au niveau du modèle, genre
validators=[validate_template]
?
Done :)
Mis à jour par Valentin Deniaud il y a presque 2 ans
Dernier truc, cette modif va générer une nouvelle migration inutile. Dans ce cas là notre pratique est d'aller modifier la dernière migration en date pour que Django n'y voit que du feu, plutôt que d'en inclure une nouvelle.
Mis à jour par A. Berriot il y a presque 2 ans
Mis à jour par A. Berriot il y a presque 2 ans
Valentin Deniaud a écrit :
Dernier truc, cette modif va générer une nouvelle migration inutile. Dans ce cas là notre pratique est d'aller modifier la dernière migration en date pour que Django n'y voit que du feu, plutôt que d'en inclure une nouvelle.
ah oui, j'avais oublié ce comportement de Django. Normalement mon dernier patch règle cela :)
Mis à jour par A. Berriot il y a presque 2 ans
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
Perfecto
Mis à jour par A. Berriot il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit e7db96aa4f8d53ff2b2d6ea2f6b2a2f5217b5eaa Author: Agate <aberriot@entrouvert.com> Date: Tue Jul 19 09:28:35 2022 +0200 opengis: validate indexing template to avoid later crashes (#67381)
Mis à jour par Transition automatique il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
opengis: validate indexing template to avoid later crashes (#67381)