Conversation
|
Hmm, there's some caveats when it comes to removing the password limit, especially with password managers. When filling in the password using a password manager for a password greater than 60 characters, it throws an error. When this happens, it just throws a For more context, this is the given issue that was opened a while ago for this specific issue: #52 (comment) If we go forward with this route, we need to think of ways to handle this situation better so that this issue doesn't pop up again! |
|
I commented on Lemmy, and I'm hoping we can hear back from snowe about the questions I asked. 😊 For now, I'll leave this as a draft. |
|
Since we never heard back from snowe and we have a suspicion that his login would have worked fine due to the known truncation behavior, I'm going to close this PR. I've leave the branch around just in case (although it would be easy enough to recreate if needed). |
Pull Request Description
Removes the limit for password length when logging in.
Issue Being Fixed
Issue Number: https://programming.dev/post/3448277
Screenshots / Recordings
Checklist
semanticLabels where applicable for accessibility?