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

Reset password. The app allows saving passwords that do not meet the requirements. #268

Open
TetianaPerinha opened this issue Apr 17, 2024 · 0 comments · Fixed by #280
Open
Assignees
Labels
bug Something isn't working high priority High priority issue

Comments

@TetianaPerinha
Copy link
Collaborator

TetianaPerinha commented Apr 17, 2024

Description: According to the requirements password must be at least 8 characters long and contain at least one letter, one digit, and may include special characters @$!%*?&.

Precondition: User is registered

Step to reproduce:

Execute the POST request as below. Replace code and email with valid data.

curl --location 'http://0.0.0.0:8083/api/v1/auth/password/change'
--header 'Content-Type: application/json'
--data-raw '{
"email": "email to resent password",
"code": "code to reset password",
"password": "t"
}'

Expected result: App should reject password, HTTP status code = 400, error message = ”Password must be at least 8 characters long and contain at least one letter, one digit, and may include special characters @$!%*?&")”.

Actual result: The app allows saving password that do not meet requirements, HTTP status code = 200

image
image

@TetianaPerinha TetianaPerinha added bug Something isn't working high priority High priority issue labels Apr 17, 2024
@nnick44 nnick44 self-assigned this Apr 21, 2024
@Sunagatov Sunagatov reopened this Apr 23, 2024
@TetianaPerinha TetianaPerinha added the to-be-tested Bug is fixed, but needs to be tested label Apr 25, 2024
@TetianaPerinha TetianaPerinha removed the to-be-tested Bug is fixed, but needs to be tested label May 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working high priority High priority issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants