Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Plus de déconnexion après modification du mot de passe #6424

Merged
merged 3 commits into from
Dec 5, 2022

Conversation

Situphen
Copy link
Member

Actuellement, on est déconnecté lorsqu'on change son mot de passe, ce qui est assez perturbant. Cette PR corrige ce soucis en utilisant la fonction de Django update_session_auth_hash prévue pour ça.

À noter pour le futur que Django propose des vues prêtes à l'emploi pour le changement et la réinitialisation du mot de passe, si on souhaite éviter de gérer cela par nous même comme actuellement.

QA : Modifier son mot de passe et vérifier que l'on n'est pas déconnecté

@Situphen Situphen added the C-Back Concerne le back-end Django label Nov 10, 2022
@coveralls
Copy link

coveralls commented Nov 10, 2022

Coverage Status

Coverage remained the same at 88.294% when pulling 1fbec26 on Situphen:no-log-out into e2633f1 on zestedesavoir:dev.

@DonKnacki
Copy link

non, je reste déconnecté après changement de mot de passe avec un compte classique (testé sur la beta et en local)
Voici mon test :

  1. connection un compte qui nécessite login + password (user2 en local)
  2. sur la page des comptes (http://localhost:8000/membres/parametres/compte/ en local) , la partie mot de passe, je rempli les 3 champs nécessaires (Mot de passe actuel, Nouveau mot de passe, confirmation) puis Enregistrer
  3. je suis rediriger automatiquement sur la page de connexion et non connecté évidemment

@Situphen
Copy link
Member Author

C'est normal que cela ne fonctionne pas sur la bêta car cette PR n'est pas sur la bêta. Par contre, c'est très étonnant que ça ne fonctionne pas chez toi en local ! Es-tu sûr d'être sur ma branche et non pas sur upstream/dev ?

Copy link
Member

@philippemilink philippemilink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ça fonctionne.

La raison de l'utilisation de la fonction n'est pas vraiment claire juste à partir du nom de la fonction, donc un petit commentaire ne fera pas de mal.

Si tu es motivé, tu peux aussi rajouter un petit test rapide.

Avec en plus un rebase, et je merge.

zds/member/views/profile.py Show resolved Hide resolved
Situphen and others added 2 commits December 5, 2022 23:27
Copy link
Member

@philippemilink philippemilink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rapport de QA

QA OK ✔️

@philippemilink philippemilink enabled auto-merge (squash) December 5, 2022 22:30
@philippemilink philippemilink merged commit a40bc42 into zestedesavoir:dev Dec 5, 2022
@Situphen Situphen deleted the no-log-out branch December 5, 2022 22:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Back Concerne le back-end Django
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants