this post was submitted on 24 Sep 2023
13 points (84.2% liked)

Thunder App

2813 readers
15 users here now

An open-source, cross-platform Lemmy client for iOS and Android.

This community is intended to discuss features and feature suggestions for Thunder; as well as friendly, respectful talks about Lemmy in general.

Please use the GitHub repository linked below to submit bug reports, so keeping track of them is easier, and make sure to search first if you already can find an issue for your report.

If there are any developers who would like to contribute, feel free to reach out on GitHub!

General Links
Website: Link
GitHub Repository: Link
Matrix Space: Link

Android Releases
IzzyOnDroid: Link
Google Play: Link

iOS Releases
Apple App Store: Link
TestFlight Beta: Link

Related Communities
Nightly Community: Link

founded 1 year ago
MODERATORS
 

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.

all 19 comments
sorted by: hot top controversial new old
[–] JoMomma@lemm.ee 11 points 1 year ago (2 children)

Why in the f do you need 60+ chars for a password?

[–] woelkchen@lemmy.world 8 points 1 year ago (1 children)

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.

[–] JoMomma@lemm.ee 3 points 1 year ago (2 children)

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

[–] woelkchen@lemmy.world 3 points 1 year ago (1 children)

Why make excuses for maximum password lengths? Just let people decided on their own if they want 200 character passwords or not.

[–] JoMomma@lemm.ee 2 points 1 year ago (1 children)

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

[–] woelkchen@lemmy.world 2 points 1 year ago

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.

[–] amju_wolf@pawb.social 2 points 1 year ago

Have you heard about pass phrases ?

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.

[–] snowe@programming.dev 3 points 1 year ago

Because there’s no reason not to and plenty of reason to?

[–] shortwavesurfer@monero.town 5 points 1 year ago (1 children)

I think 60 is the default max length set by the lemmy source code.

[–] snowe@programming.dev 3 points 1 year ago (2 children)

Mine is 100+. As far as I remember, there is no limit set for admin passwords.

[–] slazer2au@lemmy.world 6 points 1 year ago (1 children)

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.

[–] snowe@programming.dev 3 points 1 year ago

Gotcha, well admin passwords are created without using the interface, so it would not be affected by the frontend limits anyway.

[–] darklightxi@lemmy.world 3 points 1 year ago (1 children)

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

[–] snowe@programming.dev 0 points 1 year ago (1 children)

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.

[–] darklightxi@lemmy.world 1 points 1 year ago

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.

[–] micahmo@programming.dev 1 points 1 year ago (1 children)

Hi all,

Continuing the discussion from GitHub... Here is the PR I opened.

https://github.com/thunder-app/thunder/pull/766

However, there's still some discussion to be had about whether this is the right change.

@snowe, a couple questions...

Thanks!!

[–] darklightxi@lemmy.world 3 points 1 year ago

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

[–] shortwavesurfer@monero.town 1 points 1 year ago

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. 😊