Development #66964
auth_saml : KeyError: 'ENTITY_ID'
0%
Description
https://sentry.entrouvert.org/entrouvert/publik/issues/62130/
KeyError: 'ENTITY_ID' (6 additional frame(s) were not displayed) ... File "django/views/decorators/cache.py", line 44, in _wrapped_view_func response = view_func(request, *args, **kwargs) File "authentic2/views.py", line 430, in login utils_misc.get_authenticator_method(authenticator, 'login', parameters) File "authentic2/utils/misc.py", line 215, in get_authenticator_method content = response = getattr(authenticator, method)(**parameters) File "authentic2_auth_saml/models.py", line 211, in login return views.login(request, self, *args, **kwargs) File "authentic2_auth_saml/views.py", line 17, in login request, login_url='mellon_login', params={'entityID': authenticator.settings['ENTITY_ID']}
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Paul Marillonnet il y a presque 2 ans
- Projet changé de Suivi des traces à Authentic 2
- Sujet changé de KeyError: 'ENTITY_ID' à auth_saml : KeyError: 'ENTITY_ID'
suite à la migration de l’authentificateur saml en base, accessible via le BO.
Mis à jour par Paul Marillonnet il y a presque 2 ans
- Lié à Development #65483: configuration des méthodes d'authentification, SAML ajouté
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Fichier 0001-auth_saml-fix-loading-idp-settings-66964.patch 0001-auth_saml-fix-loading-idp-settings-66964.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
J'avais fait une petite modif sans conséquence que je pensais testée :/ ce bug est visible sur mon instance locale...
Le bug c'est qu'on charge les settings ajoutés par mellon via load_idp (notamment ENTITY_ID) mais ensuite on retape dans la property et donc on ne chope pas ces settings complétés.
Il n'y a pas de test sur la vue login, sûrement à raison parce que je n'arrive pas à en écrire un.
En revanche, le bug est visible sur la vue autorun, actuellement masqué par un monkeypatch de trop haut niveau. Donc je fixe pour login & autorun et je teste juste pour autorun en corrigeant ce monkeypatch.
Mis à jour par Paul Marillonnet il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
Ok. On fera un ticket pour un test sur la vue login, passons ça en attendant.
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Dupliqué par Bug #66973: KeyError: 'ENTITY_ID' ajouté
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 083f66aab5f1ae2e5043b959af5448dac9164134 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Mon Jul 4 18:37:01 2022 +0200 auth_saml: fix loading idp settings (#66964)
Mis à jour par Transition automatique il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
auth_saml: fix loading idp settings (#66964)