Bug #25655
un usager sans compte FC ne peut plus changer son mot de passe
0%
Description
(via #25654)
Dans #25535 :
+ def a2_hook_user_can_reset_password(self, user): + return user.fc_accounts.exists()
Ça fait qu'un usager qui n'a pas de compte FC ne peut plus changer son mot de passe. (???) (ou je rate un gros morceau)
Fichiers
Révisions associées
don't alter can_reset_password flag for users without an FC account (#25655)
Historique
Mis à jour par Frédéric Péters il y a plus de 5 ans
(sur le Saas de prod pour débloquer les choses je viens de modifier ce code pour faire "return True" ...).
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Fichier 0001-don-t-alter-can_reset_password-flag-for-users-withou.patch 0001-don-t-alter-can_reset_password-flag-for-users-withou.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Quand pas de compte FC, ça renvoie None, pour laisser get_user_flag continuer sa route (vérif par rapport à l'OU).
Mis à jour par Thomas Noël il y a plus de 5 ans
J'ai la même lecture que toi et ton patch me semble bon... mais je préférerai qu'un tiers sur-valide.
Car le test ajouté par #25535 n'aurait pas dû passer.
Mis à jour par Frédéric Péters il y a plus de 5 ans
Car le test ajouté par #25535 n'aurait pas dû passer.
Il me semble qu'il échouerait si models.FcAccount.objects.create(user=user, sub='xxx', token='aaa') n'était pas présent; qu'en l'état il ne joue pas la situation où utilisateur existe sans liaison FC.
Mis à jour par Thomas Noël il y a plus de 5 ans
Je dois rater un truc simple, parce que
def test_password_reset(app, mailoutbox): user = User.objects.create(email='john.doe@example.com') response = app.get('/login/') response = response.click('Reset it!').maybe_follow()
pour moi l'utilisateur existe ici sans liaison FC, et on a quand même le "Reset it" qui marche... non ? (ne jamais partir en vacances)
Mis à jour par Frédéric Péters il y a plus de 5 ans
Oui c'est au retour, après avoir cliqué sur le lien, que ça foire.
Mis à jour par Thomas Noël il y a plus de 5 ans
(largué je suis, je cède la place à un tiers, comme dit plus haut)
Mis à jour par Thomas Noël il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Thomas Noël il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit c21a16108a1065437b6271fadc09023fd0c4a660 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Mon Aug 13 11:48:13 2018 +0200 don't alter can_reset_password flag for users without an FC account (#25655)
Mis à jour par Thomas Noël il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
publié en hotfix, installation sur les prod.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Solution déployée à Fermé
don't alter can_reset_password flag for users without an FC account (#25655)