pglombardo / PasswordPusher

๐Ÿ” Securely share sensitive information with automatic expiration & deletion after a set number of views or duration. Track who, what and when with full audit logs.
https://docs.pwpush.com
Apache License 2.0
2.12k stars 358 forks source link

"Generate Password" creates wrong password length #1127

Open Jocky92 opened 1 year ago

Jocky92 commented 1 year ago

๐Ÿ› Bug Report

The generation of the passwords does not work properly. It makes little sense that the maximum length of the syllable may be shorter than the minimum length. The problem could be solved if the maximum length is checked to be longer than the minimum length. Alternatively, the other value could be adjusted automatically when a change is made.

I don't know the process of password generation, but it doesn't make much sense that way. The same problem appears with other number constellations if the distance is always only 1.

๐Ÿ”ฌ How To Reproduce

click generate Password and chose this settings: Number of Syllables: 5 Minimum Syllable Length: 4 Maximum Syllable Length: 3

Environment

Where are you running/using Password Pusher?

If applicable, what version of Password Pusher? latest Version

Screenshots

image

github-actions[bot] commented 1 year ago

Hello @Jocky92, thanks for contributing to the Password Pusher community! We will respond as soon as possible.

pglombardo commented 1 year ago

Hi @Jocky92 thanks for pointing this out. The generator definitely needs some improvements. I'll fix this on the next update to the generator code.