Development #52638
ldap: user is not LDAPUser after a password reset
0%
Description
Après avoir reset son mot de passe, la session d'un utilisateur authentifié par ldap contient un utilisateur de type User (et donc sans les attributs attendus) au lieu d'un utilisateur de type LDAPUser.
C'est du au fait que la fonction simulate_authentication remplace l'utilisateur https://git.entrouvert.org/authentic.git/tree/src/authentic2/utils/__init__.py#n1158
Fichiers
Révisions associées
Historique
Mis à jour par Loïc Dachary il y a environ 3 ans
- Fichier 0001-ldap-user-is-not-LDAPUser-after-a-password-reset.patch 0001-ldap-user-is-not-LDAPUser-after-a-password-reset.patch ajouté
Voir aussi le fil de discussion sur l'exploration de code correspondant. Et un reproducer qui montre le problème.
Mis à jour par Benjamin Dauvergne il y a environ 3 ans
- Fichier 0001-utils-set-LDAP-auth-backend-when-resetting-password-.patch 0001-utils-set-LDAP-auth-backend-when-resetting-password-.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Avec un test.
Mis à jour par Benjamin Dauvergne il y a environ 3 ans
- Fichier 0001-utils-set-LDAP-auth-backend-when-resetting-password-.patch 0001-utils-set-LDAP-auth-backend-when-resetting-password-.patch ajouté
En reprenant ton test.
Mis à jour par Serghei Mihai il y a environ 3 ans
- Statut changé de Solution proposée à En cours
J'ai l'impression que reset_password_ldap_user
prend la fixture settings
en paramètre et non la valeur transmise par test_reset_password_ldap_user
.
Mis à jour par Benjamin Dauvergne il y a environ 3 ans
- Fichier 0001-utils-set-LDAP-auth-backend-when-resetting-password-.patch 0001-utils-set-LDAP-auth-backend-when-resetting-password-.patch ajouté
- Statut changé de En cours à Solution proposée
Il manquait un ajustement aux tests.
Mis à jour par Serghei Mihai il y a environ 3 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Benjamin Dauvergne il y a environ 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 0a880ddde47dfb108bca6b91e637a705c434709f Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Thu Apr 1 17:06:39 2021 +0200 utils: set LDAP auth backend when resetting password of an LDAP account (#52638)
Mis à jour par Serghei Mihai il y a environ 3 ans
- Fichier 0001-misc-apply-black-52638.patch 0001-misc-apply-black-52638.patch ajouté
- Statut changé de Résolu (à déployer) à En cours
Ack trop rapide. La validation black ne passe pas.
Mis à jour par Benjamin Dauvergne il y a environ 3 ans
- Statut changé de En cours à Résolu (à déployer)
Pousser par ailleurs :
commit a9b667fd894668c616ad5964816fb03b88d2c49e Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Fri Apr 2 11:30:08 2021 +0200 fix code-style
Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Benjamin Renard il y a environ 3 ans
Benjamin Dauvergne a écrit :
Pousser par ailleurs :
[...]
Yes, désolé, je viens de voir l'utilisation de black et isort, j'en prends note pour mes prochains patchs.
utils: set LDAP auth backend when resetting password of an LDAP account (#52638)