Projet

Général

Profil

Development #61430

Passer ?next= à la vue de déconnexion de django-mellon

Ajouté par Benjamin Dauvergne il y a environ 2 ans. Mis à jour il y a environ 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
04 février 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Je constate que le paramètre ?next= n'est pas conservé sur un logout.


Fichiers


Demandes liées

Lié à django-mellon - Development #61431: logout: next_url n'est pas conservéFermé04 février 2022

Actions
Lié à Hobo - Development #61432: context_processors: sur une déconnexion, toujours rediriger vers la racine du siteFermé04 février 2022

Actions

Révisions associées

Révision 78a28b6b (diff)
Ajouté par Benjamin Dauvergne il y a environ 2 ans

views: pass ?next= to mellon views (#61430)

Historique

#1

Mis à jour par Benjamin Dauvergne il y a environ 2 ans

#2

Mis à jour par Frédéric Péters il y a environ 2 ans

  • Sujet changé de Passé ?next= à la vue de déconnexion de django-mellon à Passer ?next= à la vue de déconnexion de django-mellon

Perso je pense tout à fait normal qu'une déconnexion envoie ensuite vers l'accueil; conserver la page actuelle (si c'est bien ce qui est proposé ici), ça va être régulièrement se manger une page d'erreur accès interdit au retour, ou une redirection vers l'écran de connexion.

#3

Mis à jour par Benjamin Dauvergne il y a environ 2 ans

#4

Mis à jour par Benjamin Dauvergne il y a environ 2 ans

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

Perso je pense tout à fait normal qu'une déconnexion envoie ensuite vers l'accueil; conserver la page actuelle (si c'est bien ce qui est proposé ici), ça va être régulièrement se manger une page d'erreur accès interdit au retour, ou une redirection vers l'écran de connexion.

Je n'ai pas de souci à être d'accord avec toi, mais dans ce cas je préfère que ce choix soit manifeste dans user_urls pas via un oubli dans la vue de logout elle même. Le cas d'usage typique ce serait un lien "Vous n'êtes pas Jean Darmette, changer d'utilisateur...", où on voudra revenir sur une page nécessitant une connexion après une déconnexion.

Je vais adapter le patch proposer un changement coté hobo.

#5

Mis à jour par Benjamin Dauvergne il y a environ 2 ans

  • Lié à Development #61432: context_processors: sur une déconnexion, toujours rediriger vers la racine du site ajouté
#6

Mis à jour par Benjamin Dauvergne il y a environ 2 ans

À passer après #61432.

#7

Mis à jour par Paul Marillonnet il y a environ 2 ans

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

Ok.

#8

Mis à jour par Benjamin Dauvergne il y a environ 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 78a28b6be96c2e6416406e8445fee69afcabec9f
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Fri Feb 4 13:03:09 2022 +0100

    views: pass ?next= to mellon views (#61430)
#9

Mis à jour par Transition automatique il y a environ 2 ans

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

Mis à jour par Transition automatique il y a environ 2 ans

Automatic expiration

Formats disponibles : Atom PDF