Xyphyn / photon

A replacement for lemmy-ui with more features, a better design, and more customizability.
https://phtn.app
GNU Affero General Public License v3.0
311 stars 35 forks source link

Cannot login if password is longer than 60 characters. #312

Closed sevonj closed 1 month ago

sevonj commented 2 months ago

Describe the bug I gave my lemmy account a long password. I can login through other frontends (Lemmy-UI, Voyager), but not Photon. I tested quite a few passwords and found that 60 characters works, but anything longer gives "Incorrect login," with correct password.

To Reproduce (if applicable) Steps to reproduce the behavior:

  1. Change password in Lemmy-UI, to 61 characters or longer
  2. Try to login via Photon

Expected behavior A successful login.

Screenshots Screenshot from 2024-05-15 14-38-19

Versions

sevonj commented 2 months ago

Well I found out what's actually going on, 5 minutes after filing this issue.

Max password length is 60 characters. The input fields on lemmy-ui silently truncated my passwords.

I guess the issue is now reduced to a suggestion: Limit login password field to 60 characters and maybe let the user know if they're trying to input a too long password.

neatchee commented 2 months ago

Well I found out what's actually going on, 5 minutes after filing this issue.

Max password length is 60 characters. The input fields on lemmy-ui silently truncated my passwords.

I guess the issue is now reduced to a suggestion: Limit login password field to 60 characters and maybe let the user know if they're trying to input a too long password.

Thank you for posting this comment. This just saved me after HOURS of troubleshooting. Slightly annoyed that my password got truncated and every other application just silently handled it 🤷