Bug #5895
incompatibilité /auth ou /tryauth avec une querystring (?cancelurl=...)
0%
Description
Quand on utilise /tryauth, on ne peut pas envoyer de querystring (par exemple de ?cancelurl=...).
(C'est génant mais pas bloquand sur Orléans où j'aurai aimé ajouter du tryauth sur quelques formulaires, pour rendre la navigation plus limpide, i.e. éviter d'arriver sur un wcs sans y être connecté alors qu'on a une session sur l'idp, bla bla bla)
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 5 ans
- Dupliqué par Development #27232: Conserver les paramètres de la query string lors de l'appel d'un formulaire avec /tryauth ajouté
Mis à jour par Frédéric BISSON il y a presque 4 ans
Bonjour,
Ce bug ne semble pas avoir été corrigé et est encore à l'état nouveau.
Est-il prévu de le corriger ?
Merci
Cordialement
Frédéric BISSON
Mis à jour par Thomas Noël il y a presque 4 ans
Frédéric BISSON a écrit :
Ce bug ne semble pas avoir été corrigé et est encore à l'état nouveau.
Est-il prévu de le corriger ?
Il ne gêne pas de fonctionnements sur les Publik actuellement en production, donc il n'y a pas eu de priorisation de ce bogue. Si vous avez un cas d'usage qui nécessite une correction, faites-nous signe (typiquement par un ticket sur votre projet) : nous regarderons et vous dirons ce qui est faisable.
Mis à jour par Serghei Mihai il y a presque 4 ans
A la Dodrogne ça construit des URLs vers les formulaires à la main, genre {{eservices_url}}contact/contact-sve/?theme=culture (cf #42701). Le besoin serait plutôt de faire {{eservices_url}}contact/contact-sve/tryauth?theme=culture pour assurer l'authentification passive, sans perdre la querystring.
Mis à jour par Serghei Mihai il y a presque 4 ans
- Lié à Development #12867: Implémenter l'authentification passive automatique ajouté
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Dupliqué par Development #56813: tryauth & get params ajouté
Mis à jour par Lauréline Guérin il y a plus de 2 ans
- Fichier 0002-misc-auth-tryauth-forceauth-with-get-params-5895.patch 0002-misc-auth-tryauth-forceauth-with-get-params-5895.patch ajouté
- Fichier 0001-misc-split-auth-tests.patch 0001-misc-split-auth-tests.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Lauréline Guérin il y a plus de 2 ans
- Lié à Development #56812: Caluire & cellules avec lien vers une démarche ajouté
Mis à jour par Thomas Noël il y a plus de 2 ans
« self.formdef.get_url() + '?' + get_request().get_query() » : en cas de query string vide, ça ne va pas ajouter un "?" dans l'URL finale ? Si non, peux-tu laisse le test simple resp = get_app(pub).get('/test/auth' qui prouvera que non ?
Mis à jour par Lauréline Guérin il y a plus de 2 ans
Mis à jour par Thomas Noël il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Lauréline Guérin il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 652a2c7a08bbadf9175b5426b7c6bf56ff5e5913
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date: Fri Sep 10 10:02:36 2021 +0200
misc: auth/tryauth/forceauth with get params (#5895)
commit 7481ab86d383af33bc554bc0446c1acf50914867
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date: Fri Sep 10 09:52:56 2021 +0200
misc: split auth tests
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
misc: auth/tryauth/forceauth with get params (#5895)