Bug #57755
braine l’alleud, ne pas échapper l'url des délibérations communales
0%
Description
La requête fait par la cellule "deliberations-communales" utilise une variable hobo qui contient des '&'.
Les '&' sont échappés en '&' dans la requête qui ne fonctionne pas comme attendu.
Fichiers
Révisions associées
Historique
Mis à jour par Nicolas Roche il y a plus de 2 ans
- Fichier 0001-braine-l-alleud-do-not-autoescape-url-on-deliberatio.patch 0001-braine-l-alleud-do-not-autoescape-url-on-deliberatio.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Nicolas Roche il y a plus de 2 ans
Vu https://dev.entrouvert.org/issues/56545#note-23
J'ai oublié de noter que ça avait été intégré poussé déployé etc.
J'ai l'impression que ce patch a été laissé de côté
$ ssh combo.node2.test.saas.entrouvert.org $ grep autoescape /usr/share/publik/themes/publik-base/templates/variants/braine-l-alleud/combo/json/deliberations-communales.html
Mis à jour par Frédéric Péters il y a plus de 2 ans
Oui dans l'autre ticket je parlais de l'autre ticket.
Mis à jour par Frédéric Péters il y a plus de 2 ans
Je pense qu'ici est à laisser de côté, que l'accès aux délibérations doit être revu pour exploiter le connecteur plone rest api; ici c'était juste un truc rapide pour dépanner un moment.
Mis à jour par Nicolas Roche il y a plus de 2 ans
Oui, tout à fait.
Cependant l'instance Plone sur laquelle on va chercher les délibérations nous impose de nous authentifier,
et elle n'a pas encore été mise à jour pour proposer la connexion OIDC.
Plutôt que de gérer les 2 modes d'authentification dans le connecteur (et faire une usine à gaz), je me suis dit que l'on pourrait différer l'utilisation du connecteur ici.
Mis à jour par Frédéric Péters il y a plus de 2 ans
Cependant l'instance Plone sur laquelle on va chercher les délibérations nous impose de nous authentifier,
J'ai créé #57978.
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Solution proposée à Résolu (à déployer)
J'ai refait en mettant juste |safe.
commit d64f74bbf60f3b061f66db6019c1eb1058ece367 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Tue Oct 26 09:06:10 2021 +0200 braine-l-alleud: do not escape URL to deliberations-communales (#57755)
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
braine-l-alleud: do not escape URL to deliberations-communales (#57755)