Projet

Général

Profil

Bug #57755

braine l’alleud, ne pas échapper l'url des délibérations communales

Ajouté par Nicolas Roche il y a plus de 2 ans. Mis à jour il y a plus de 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
12 octobre 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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

Révision d64f74bb (diff)
Ajouté par Frédéric Péters il y a plus de 2 ans

braine-l-alleud: do not escape URL to deliberations-communales (#57755)

Historique

#1

Mis à jour par Nicolas Roche il y a plus de 2 ans

#3

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

#4

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.

#5

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.

#6

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.

#7

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.

#8

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)
#9

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

Formats disponibles : Atom PDF