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

Redirection lors du renouvelement de mdp #1810

Closed
TheoLechemia opened this issue Mar 30, 2022 · 1 comment
Closed

Redirection lors du renouvelement de mdp #1810

TheoLechemia opened this issue Mar 30, 2022 · 1 comment
Assignees
Labels

Comments

@TheoLechemia
Copy link
Member

TheoLechemia commented Mar 30, 2022

Version
2.9.2

Description du bug
Lorsqu'une demande un nouveau mot de passe est réalisée, un mail est envoyé avec un lien pour renouveler celui-ci. Ex. de lien : http://127.0.0.1:4200/#/new-password?token=63430871539009728421038202709064474463

À cause du APP_INIALIZER un appel est fait sur /modules et comme l'utilisateur n'a pas de token à ce moment là, il est redirigé vers /login sans pouvoir changer son mot de passe.

Comportement attendue
Ne pas être redirigé vers la page de login et pouvoir changer son mdp.

Comment reproduire
Demander un nouveau mdp et ne pas avoir de token en cookie.

Solution
Il faudrait sortir les pages /login, /inscription et /new-password du module principale pour ne pas avoir l'APP_INIALIZER sur ces pages. Pour éviter l'affichage des messages d'erreur liés au code 401 sur ces pages, il faudrait les associer au login. Nous aurions donc les chemins : /login, /login/inscription et /login/new-password

@camillemonchicourt
Copy link
Member

Cela explique le retour ici il me semble : #1620

@jpm-cbna jpm-cbna self-assigned this Apr 2, 2022
jpm-cbna added a commit that referenced this issue Apr 2, 2022
Load login module with the same method of other module.
Group sign-up, new-password and login components.
Fix display of 401 error on all login module pages.

Closes #1810, fix #1620.
jpm-cbna added a commit that referenced this issue Apr 2, 2022
Load login module with the same method of other module.
Group sign-up, new-password and login components.
Fix display of 401 error on all login module pages.

Closes #1810, fix #1620.
jpm-cbna added a commit that referenced this issue Apr 5, 2022
Load login module with the same method of other module.
Group sign-up, new-password and login components.
Fix display of 401 error on all login module pages.

Closes #1810, fix #1620.
jpm-cbna added a commit that referenced this issue Apr 5, 2022
Load login module with the same method of other module.
Group sign-up, new-password and login components.
Fix display of 401 error on all login module pages.

Closes #1810, fix #1620.
jpm-cbna added a commit that referenced this issue Apr 6, 2022
Load login module with the same method of other module.
Group sign-up, new-password and login components.
Fix display of 401 error on all login module pages.

Closes #1810, fix #1620.
jpm-cbna added a commit that referenced this issue Oct 11, 2022
Load login module with the same method of other module.
Group sign-up, new-password and login components.
Fix display of 401 error on all login module pages.

Closes #1810, fix #1620.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants