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

Password Change Not Validating Old Password #12584

Closed
arudra123 opened this issue Jul 25, 2023 · 6 comments · Fixed by #14273 or #14682
Closed

Password Change Not Validating Old Password #12584

arudra123 opened this issue Jul 25, 2023 · 6 comments · Fixed by #14273 or #14682
Assignees
Labels
backend UI UI specific issues

Comments

@arudra123
Copy link

In the user profile, system is not validating the old password entered by the user when they attempt to change their password. This means that a user can change their password to any value, even if it is the same as their old password.
Steps to Reproduce:
Log in to the user profile system.
Click on the "Change Password" link.
Enter your old password.
Enter a new password.
Enter the new password again to confirm.
Click on the "Update Password" button.
Expected Results:
The system should validate the old password and only allow the user to change their password if the old password is correct.

@harshach harshach moved this to Backend - Bugs & Minor Features in Release 1.3.0 Oct 22, 2023
@mohityadav766
Copy link
Contributor

@chirag-madlani can you please assign someone from UI , if the User is updating his own account UI needs to send ChangePasswordRequest SELF for requestType , currently Admins of the system have the privilege to update anyone's password , so for their own password if don't send SELF it won't require the OLD_PASSWORD since the user is already an admin, so please send SELF in case the user updates his own account

@chirag-madlani
Copy link
Collaborator

@mohityadav766 assigned this to @Ashish8689.
Thanks

@chirag-madlani chirag-madlani added the UI UI specific issues label Nov 20, 2023
@harsh-vador
Copy link
Contributor

I will take this up

@harshach
Copy link
Collaborator

we are also limiting only certain special characters and max length is set to 16 chars. We need to increase it cc @mohityadav766 @chirag-madlani

@chirag-madlani
Copy link
Collaborator

@harshach will rule with @mohityadav766 for this ticket only

@mohityadav766
Copy link
Contributor

closing this it handled

@harshach harshach moved this from UI - Bugs & Minor Features to Done in Release 1.3.0 Jan 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend UI UI specific issues
Projects
No open projects
Status: Done
7 participants