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.

Sign up for GitHub

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

Jump to bottom

Fix the broken LoginAttemptsBeforeLockout #6140

Merged
merged 1 commit into from
Sep 30, 2024
Merged

Fix the broken LoginAttemptsBeforeLockout #6140

merged 1 commit into from
Sep 30, 2024

Conversation

nyanmisaka
Copy link
Member

Changes

  • Fix the broken LoginAttemptsBeforeLockout

Issues

  • fixes a typo made in be891c3, to reproduce:
  1. Upgrade to 10.10
  2. Save settings in Dashboard->User
  3. Happened to enter the wrong admin password once when logging in
  4. You are locked out of your server

1

@nyanmisaka
Fix the broken LoginAttemptsBeforeLockout
f341b11 
fixes a typo made in be891c3

Signed-off-by: nyanmisaka <nst799610810@gmail.com>
@nyanmisaka nyanmisaka requested a review from a team as a code owner September 30, 2024 14:07
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Sep 30, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@jellyfin-bot
Copy link
Collaborator

Cloudflare Pages deployment

Latest commit f341b11c96a66cf62b5ded9e7d1be0a66cfd632d
Status ✅ Deployed!
Preview URL https://8ad69cef.jellyfin-web.pages.dev
Type 🔀 Preview

@thornbill thornbill merged commit 84363a8 into jellyfin:master Sep 30, 2024
12 checks passed
@nyanmisaka nyanmisaka deleted the fix-LoginAttemptsBeforeLockout branch September 30, 2024 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thornbill thornbill thornbill approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@nyanmisaka @jellyfin-bot @thornbill