Project

General

Profile

Development #52638

ldap: user is not LDAPUser after a password reset

Added by Loïc Dachary about 3 years ago. Updated about 3 years ago.

Status:
Fermé
Priority:
Normal
Category:
LDAP
Target version:
-
Start date:
01 April 2021
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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


Files

Associated revisions

Revision 0a880ddd (diff)
Added by Benjamin Dauvergne about 3 years ago

utils: set LDAP auth backend when resetting password of an LDAP account (#52638)

History

#2

Updated by Benjamin Dauvergne about 3 years ago

Avec un test.

#4

Updated by Serghei Mihai about 3 years ago

  • Status changed from Solution proposée to 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.

#5

Updated by Benjamin Dauvergne about 3 years ago

Il manquait un ajustement aux tests.

#6

Updated by Serghei Mihai about 3 years ago

  • Status changed from Solution proposée to Solution validée
#7

Updated by Benjamin Dauvergne about 3 years ago

  • Status changed from Solution validée to 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)
#8

Updated by Serghei Mihai about 3 years ago

Ack trop rapide. La validation black ne passe pas.

#9

Updated by Benjamin Dauvergne about 3 years ago

  • Status changed from En cours to 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

#10

Updated by Frédéric Péters about 3 years ago

  • Status changed from Résolu (à déployer) to Solution déployée
#11

Updated by Benjamin Renard about 3 years ago

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.

Also available in: Atom PDF