Development #62723
correction à l'url de logout renseignée pour wcs
0%
Description
Depuis #61191 l'url de déconnexion est définie par la variable logout_url
, posée dans le contexte des pages par hobo et a la valeur /logout/?next=...
.
Or sur wcs la vue de déconnexion est sur l'url /logout
.
Lorsqu'un usager se trouvant sur une page front wcs clique sur Déconnexion
il obtient une page 404.
Fichiers
Révisions associées
context processor: adapt to use existing logout URL for w.c.s. (#62723)
Historique
Mis à jour par Serghei Mihai il y a environ 2 ans
- Fichier 0001-context_processors-define-logout_url-var-62723.patch 0001-context_processors-define-logout_url-var-62723.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
IMO wcs devrait définir la variable avec la bonne url.
A voir si on balance un next=
pour permettre à l'usager de revenir sur la page du formulaire sans être connecté.
Mis à jour par Serghei Mihai il y a environ 2 ans
- Fichier 0001-root-add-support-for-trailing-slash-on-logout-url.patch ajouté
Une autre option serait de prendre en compte l'url avec le / final.
Mis à jour par Serghei Mihai il y a environ 2 ans
- Fichier
0001-root-add-support-for-trailing-slash-on-logout-url.patchsupprimé
Mis à jour par Serghei Mihai il y a environ 2 ans
Mis à jour par Thomas Noël il y a environ 2 ans
C'est normal la suppression de « saml = saml2.Saml2Directory() » ?
Aussi je crois que ça manque un peu de tests pour vérifier que /logout et /logout/ fonctionnent bien tous les deux pareil
Mis à jour par Serghei Mihai il y a environ 2 ans
- Fichier 0001-root-add-trailing-slash-to-logout-url-62723.patch 0001-root-add-trailing-slash-to-logout-url-62723.patch ajouté
- Tracker changé de Bug à Development
- Sujet changé de mauvaise valeur de la variable "logout_url" à ajouter le / final dans l'url de déconnexion
Thomas Noël a écrit :
C'est normal la suppression de « saml = saml2.Saml2Directory() » ?
Je pensais, à tort, que c'était pas utilisé ailleurs.
Aussi je crois que ça manque un peu de tests pour vérifier que /logout et /logout/ fonctionnent bien tous les deux pareil.
Tests mis à jour.
Mis à jour par Frédéric Péters il y a environ 2 ans
Pas fan de bouger tout ça plutôt que corriger l'URL mal posée à la base; surtout qu'il y a déjà une branche spécifique wcs dans ce code dans hobo.
Mis à jour par Frédéric Péters il y a environ 2 ans
- Fichier 0001-context-processor-adapt-to-use-existing-logout-URL-f.patch 0001-context-processor-adapt-to-use-existing-logout-URL-f.patch ajouté
- Projet changé de w.c.s. à Hobo
Comme ça.
Mis à jour par Serghei Mihai il y a environ 2 ans
- Statut changé de Solution proposée à Solution validée
- Assigné à mis à Frédéric Péters
Ok, go.
Mis à jour par Frédéric Péters il y a environ 2 ans
- Sujet changé de ajouter le / final dans l'url de déconnexion à correction à l'url de logout renseignée pour wcs
- Statut changé de Solution validée à Résolu (à déployer)
commit 120723aeab00fdd41400192f2e31c76a9112c3ba Author: Frédéric Péters <fpeters@entrouvert.com> Date: Mon Mar 14 20:17:24 2022 +0100 context processor: adapt to use existing logout URL for w.c.s. (#62723)
Mis à jour par Transition automatique il y a environ 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
context processor: adapt to use existing logout URL for w.c.s. (#62723)