PhilippC / PhilippC/keepass2android
[BUG] High password length during generation crashes android app
Nobody has claimed this yet.
- Dominant language
- C#
- Stars
- 6.2k
- Forks
- 478
- Avg merge
- 1h 4m
- Merged PRs (30d)
- 2
Description
Hello,
Dont ask why but I wanted to generate a 1025 character password. I typo-ed this and added an extra 5 on the end. This 10kb password generation consistently crashes the android app. Worse still, the app remembers the desired password length and crashes the next time I want to open the dialog 😂
Please add a sensible limit to the passowrd length, or at least handle the exception / timeout more gracefully and give user feedback, and dont remember the length if this happens.
I'm currently locked out of generating a new password, hoping an app data cleans will fix it.
Thanks for a perfect app otherwise.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reproducing password generation with a 1025-character length and observe the generation dialog's handling of the saved length. Done means the Android app does not crash, gives useful feedback for an excessive length or failure, and does not retain a value that causes the dialog to crash again.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- android, csharp
- Domain
- mobile, security
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100