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

Inform user on reason for password invalidity during password reset #703

Open
Steffengreiner opened this issue Jul 10, 2024 · 2 comments · May be fixed by #819
Open

Inform user on reason for password invalidity during password reset #703

Steffengreiner opened this issue Jul 10, 2024 · 2 comments · May be fixed by #819
Assignees
Labels
bug Something isn't working simple Implementation effort stale triage An issue that is in discussion to be either scheduled for implepmentation or denied

Comments

@Steffengreiner
Copy link
Contributor

Steffengreiner commented Jul 10, 2024

Describe the bug
If an invalid password is provided as a to be set password during password reset the user is shown a general exception instead of being informed on why the provided password is invalid.

To Reproduce
Steps to reproduce the behavior:

  1. Progress the password reset process until a new password can be specified.
  2. In this New Password screen input a password string that is invalid (e.g. due to length being too short)
  3. The data manager application will show a general exception error message instead of specifying to the user why the provided password is invalid.

Expected behavior
The user is informed about the reason why the provided password is too short instead of being shown a general application error. Ideally below the password setting textfield similiar to how it's done during user registration.

Screenshots
Screenshot 2024-07-10 at 16 28 16

Desktop (please complete the following information):

  • MacOs Sonoma 14.5
  • Firefox 127.0.2 (64-Bit)
  • Build on qpylon/dev
@Steffengreiner Steffengreiner added bug Something isn't working simple Implementation effort labels Jul 10, 2024
@KochTobi
Copy link
Member

In the new password layout no validation takes place. Compare the field in the registration layout. The same validation should take place.

@Steffengreiner Steffengreiner self-assigned this Jul 12, 2024
@sven1103 sven1103 added the triage An issue that is in discussion to be either scheduled for implepmentation or denied label Aug 5, 2024
Copy link

github-actions bot commented Sep 5, 2024

This issue is stale because it has been open for 30 days with no activity.

@github-actions github-actions bot added the stale label Sep 5, 2024
@sven1103 sven1103 assigned sven1103 and unassigned sven1103 Sep 24, 2024
@wow-such-code wow-such-code self-assigned this Sep 24, 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 simple Implementation effort stale triage An issue that is in discussion to be either scheduled for implepmentation or denied
Projects
Status: No status
Status: Needs Review
4 participants