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

[Bug]: New password length requirements breaks login for users #2381

Open
4 tasks done
richardARPANET opened this issue Feb 28, 2024 · 2 comments
Open
4 tasks done

[Bug]: New password length requirements breaks login for users #2381

richardARPANET opened this issue Feb 28, 2024 · 2 comments
Labels
bug Something isn't working

Comments

@richardARPANET
Copy link

Requirements

  • Is this a bug report? For questions or discussions use https://lemmy.ml/c/lemmy_support
  • Did you check to see if this issue already exists?
  • Is this only a single bug? Do not put multiple bugs in one issue.
  • Is this a backend issue? Use the lemmy-ui repo for UI / frontend issues.

Summary

lemmy-ui:0.19.1
lemmy:0.19.1

In an older version users could register with passwords <10 chars, in the new version frontend code you can only login with 10+ chars. This means users with old shorter passwords cannot login to the board.

Please add a fix, or a config option to make the password min length shorter so these users may login. Thanks

Steps to Reproduce

As above.

Technical Details

As above.

Version

0.19.1

Lemmy Instance URL

No response

@richardARPANET richardARPANET added the bug Something isn't working label Feb 28, 2024
@dessalines
Copy link
Member

dessalines commented Feb 29, 2024

I'm not sure the best way to fix this... possibly a DB migration that fills an extra character until it hits the 10 char minimum, for passwords < 10 chars.

@richardARPANET
Copy link
Author

richardARPANET commented Feb 29, 2024

I would suggest

  1. Keep the new min chars limit on the sign up form
  2. Remove it from the UI on the login form. (If I edit the login form HTML it lets me login with a short password)

@Nutomic Nutomic transferred this issue from LemmyNet/lemmy Mar 1, 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
Projects
None yet
Development

No branches or pull requests

2 participants