Bug #7271
LoginView.sso_failure() n'utilise pas la configuration du bon idp (en général, dans nos cas particuliers ça marchait)
100%
Description
sso_failure() fait un self.get_idp(request) qui est sensé retourné l'IdP choisi par l'utilisateur via le paramètre de requête entity_id sinon il prend le premier. Ça marche bien dans notre cas car on a généralement qu'un IdP (mais pas pour u-auth par exemple).
On devrait utiliser directement utils.get_idp(login.remoteProviderId).
Fichiers
Révisions associées
views: in sso_failure() the call to self.get_id() could never work, replace by utils.get_idp()
In SSO response treatment we do not know anymore the requested IdP from the query string
we must look in the LassoLogin object.
fixes #7271
Historique
Mis à jour par Benjamin Dauvergne il y a presque 9 ans
- Fichier 0001-utils-add-a-default-return-value-to-utils.get_idp.patch 0001-utils-add-a-default-return-value-to-utils.get_idp.patch ajouté
- Fichier 0002-views-in-sso_failure-the-call-to-self.get_id-could-n.patch 0002-views-in-sso_failure-the-call-to-self.get_id-could-n.patch ajouté
Le premier patch fait renvoyer un dictionnaire vide à utils.get_idp() au cas où l'entity id de l'émetteur de la requête serait inconnu.
Le deuxième patch remplacer self.get_idp() par utils.get_idp().
Mis à jour par Thomas Noël il y a presque 9 ans
- Sujet changé de LoginView.sso_failure() n'utilise pas la configuration du bonne idp (en général, dans nos cas particuliers ça marchait) à LoginView.sso_failure() n'utilise pas la configuration du bon idp (en général, dans nos cas particuliers ça marchait)
Ack
Mis à jour par Benjamin Dauvergne il y a presque 9 ans
- Statut changé de Nouveau à Résolu (à déployer)
- % réalisé changé de 0 à 100
Appliqué par commit django-mellon|4b71bbca3a8adf892bf83d9bde6aca387dd702a5.
Mis à jour par Benjamin Dauvergne il y a plus de 8 ans
- Statut changé de Résolu (à déployer) à Fermé
utils: add a default return value to utils.get_idp()
refs #7271