Skip to content

Avoid fruitless login attempts - #8499

Merged
MorrisJobke merged 2 commits into
masterfrom
avoid-fruitless-login-attempts
Feb 22, 2018
Merged

Avoid fruitless login attempts#8499
MorrisJobke merged 2 commits into
masterfrom
avoid-fruitless-login-attempts

Conversation

@blizzz

Copy link
Copy Markdown
Member
  • LDAP: don't attempt to bind with last, active bind credentials
  • User Manager: don't try to login user with email ids twice if it failed (this should be move to the backend, but I need something backportable)

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

@MorrisJobkeMorrisJobke left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Makes sense and works 👍

@MorrisJobke
MorrisJobke merged commit 930aad5 into masterFeb 22, 2018
@MorrisJobke
MorrisJobke deleted the avoid-fruitless-login-attempts branch February 22, 2018 18:19
@blizzz

Copy link
Copy Markdown
MemberAuthor

Backport to 13: #8532

12: only the LoginController change applies, #8533

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@blizzz@rullzer@MorrisJobke