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

Login: Adjust error message when user exceed login attempts #70736

Merged
merged 2 commits into from
Jun 30, 2023

Conversation

RoxanaAnamariaTurc
Copy link
Contributor

What is this feature?

Adjust the error message when user exceed login attempts. Added unit tests as well to prevent regression.

Why do we need this feature?

Users know when they are blocked out of their account.

Who is this feature for?

Everyone.

Which issue(s) does this PR fix?:

Fixes #62037

Special notes for your reviewer:

Please check that:

  • It works as expected from a user's perspective.
  • If this is a pre-GA feature, it is behind a feature toggle.
  • The docs are updated, and if this is a notable improvement, it's added to our What's New doc.

@RoxanaAnamariaTurc RoxanaAnamariaTurc added add to changelog area/frontend/login Issues related to login, changing password, reset password no-backport Skip backport of PR labels Jun 27, 2023
@RoxanaAnamariaTurc RoxanaAnamariaTurc added this to the 10.1.x milestone Jun 27, 2023
@RoxanaAnamariaTurc RoxanaAnamariaTurc requested a review from a team June 27, 2023 10:01
@RoxanaAnamariaTurc RoxanaAnamariaTurc requested a review from a team as a code owner June 27, 2023 10:01
@RoxanaAnamariaTurc RoxanaAnamariaTurc requested review from tskarhed and ashharrison90 and removed request for a team June 27, 2023 10:01
@ashharrison90 ashharrison90 requested review from a team, vtorosyan and danielkenlee and removed request for a team June 27, 2023 10:14
Co-authored-by: Ashley Harrison <ashley.harrison@grafana.com>
Co-authored-by: L-M-K-B <48948963+L-M-K-B@users.noreply.github.com>
Co-authored-by: tskarhed <1438972+tskarhed@users.noreply.github.com>
Copy link
Contributor

@ashharrison90 ashharrison90 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good mob 🔱

@ashharrison90
Copy link
Contributor

@grafana/grafana-authnz-team @grafana/docs-squad any thoughts on these changes? 🤔

Copy link
Contributor

@IevaVasiljeva IevaVasiljeva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense to me, but I'll also check if anyone else from the team has any objections to exposing this information.

@IevaVasiljeva
Copy link
Contributor

The team is happy with the PR being merged. Thanks for improving the auth experience! ❤️

Copy link
Collaborator

@imatwawana imatwawana left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@ashharrison90 ashharrison90 merged commit ea361f5 into main Jun 30, 2023
13 checks passed
@ashharrison90 ashharrison90 deleted the login-blocked branch June 30, 2023 10:22
harisrozajac pushed a commit that referenced this pull request Jun 30, 2023
* Adjust error message when user exceed login attempts

* empty commit to add coauthors

Co-authored-by: Ashley Harrison <ashley.harrison@grafana.com>
Co-authored-by: L-M-K-B <48948963+L-M-K-B@users.noreply.github.com>
Co-authored-by: tskarhed <1438972+tskarhed@users.noreply.github.com>

---------

Co-authored-by: Ashley Harrison <ashley.harrison@grafana.com>
Co-authored-by: L-M-K-B <48948963+L-M-K-B@users.noreply.github.com>
Co-authored-by: tskarhed <1438972+tskarhed@users.noreply.github.com>
@ricky-undeadcoders ricky-undeadcoders modified the milestones: 10.1.x, 10.1.0 Aug 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
add to changelog area/frontend/login Issues related to login, changing password, reset password area/frontend no-backport Skip backport of PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Login: Poor user feedback when login is blocked after brute force / too many failed attempts
6 participants