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

fix: Fixed the password-reset endpoint #884

Closed

Conversation

johnson-oragui
Copy link
Collaborator

This fix corrects the the previous implementation where users are not appropriately sent a link/valid link when they request for a password change.

Description

This fix corrects the the previous implementation where users are not appropriately sent a link/valid link when they request for a password change. With the changes done, users can now receive mails regarding heir change of password.

Related Issue (Link to issue ticket)

Motivation and Context

Users/Clients should be able to changes/reset their password when they have forgotten it or can no longer sign for some security reason.

How Has This Been Tested?

This has been tested with pytest and postman.

Screenshots (if appropriate - Postman, etc):

Reset token sent to user successfully
reset passsword link sent

reset token received in user email

reset password email received

user successfully changes password
reset password successful

user tries to reuse token
reset password try to reuse token

passwords validations
reset password and confirm password not a match

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant