It seems like the password limit is set to 60 characters so I’m unable to login to my instance. There probably should be no limit in the app because each server could have different limits set.
When you say, “It seems like the password limit is set to 60 characters so I’m unable to login to my instance.”, did you try logging in with a truncated password? Or are you just assuming that, if we limit the length, it won’t work? I only ask because this Thunder discussion led us to find that truncating actually works for some users, which is what led to the original limit.
I believe Jerboa also has this character limit as mentioned in this PR. It seems like they truncate the password to take the first 60 characters if it’s too long. This is what Thunder also follows as of right now, except it’s shown explicitly to the user in the login modal
Its looking like this may be fixed. Here is a recent chat from micahmo in the thunder general chat on matrix
I Just noticed that this was posted by the snowe, the admin of programming.dev, which is the instance I use haha! Just thought it was cool that he used Thunder!
I read through that thread, and regardless of whether there’s a limit or not, or there’s an issue with the backend, etc., etc., I think it makes sense to remove the limit in Thunder. Specifically, snowe said this:
I can log in on every other app perfectly fine
If we were facilitating account creation, then sure maybe we should do a little more validation. But for login, I say let the user type whatever they want and let the backend handle it.
I’ll open a quick PR to remove this limit. Of course, up to Hamlet, if you agree. 😊
Computers get faster all the time, making brute force cracking of passwords easier all the time. Password managers don’t care how long a password is. The task of filling it out is the same.
60 character passwords with any amount of complexity would take effectively infinite time to brute, an 18 character password with complex characters would take millions of years… There is no reason to use 60, let alone more than
Well, there could be very reasonable reasons for the limit, like keeping the hash tables sane, or keeping databases from needing unnecessary padding, but there really isn’t any reasonable reason for needing 60+ characters in passwords
Well, there could be very reasonable reasons for the limit, like keeping the hash tables sane, or keeping databases from needing unnecessary padding, but there really isn’t any reasonable reason for needing 60+ characters in passwords
And how is this the duty of a client app to police that? OP says it’s about being able to log into their own instance and the client app is blocking this.
Also, none of those is a sane reason to limit password length. A huge point of hashing is having short, constant length strings on output no matter the input. There’s no limitation or database issue there.
The only reason to limit password length is actually security (for bad algorithms) and DoS, but that requires a limit in the thousands.
So what I have noticed is 60 is the max according to the source code, but if you use a password manager that fills the field in for you, the browser in my case Firefox, ignores the limit and accepts the full length password.
Hey there, like others have mentioned, I think this is a limitation on lemmy’s end which limits the password length up to 60 characters.
This is the source code for lemmy’s backend if you’re curious. If you think this is not the case, feel free to create a new issue on GitHub and we can take a further look into this! Let me know if you need any more clarification :D
I think that check must be bypassed for admin passwords, or it was instituted after I created programming.dev, because my password is 100 characters and I can log in on every other app perfectly fine. Even if that was the limit, it still should be enforced by the backend on login, not on the frontend, except for maybe initial account creation.
I think that check must be bypassed for admin passwords, or it was instituted after I created programming.dev
That could be a possibility - we can do some tests to verify if that’s the case. I found this related issue which might indicate that all auth flows through the same logic.
Even if that was the limit, it still should be enforced by the backend on login, not on the frontend
Unfortunately, that might not be the case. Dessalines mentioned in this comment that the backend doesn’t truncate overly long passwords, and throws an error instead. Although, as you mentioned, this might be bypassed for admin users.
Either way, I think we can take a deeper look at this and verify this information! Feel free to create a new issue for this on GitHub if you’re able to so that we can track this issue better.
Add comment