Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.6k
[Validator] Improve entropy estimation in PasswordStrengthValidator#49879
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Hello@nicolas-grekas, I played with this new calculation method and it gives an entropy that is approx +40% compare to the one I proposed.
|
Also, as noted by@stof in this#49868 (comment), the named argument ofconstriant should be at the end. |
7c45942
to99c09ff
Compare
Right, updated!
Thanks for checking. I think the new logic better accounts for things like mixing character case, symbols, and other and also duplicate chars. |
kriskoch commentedJun 20, 2023 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Is it just me or does this feature seem to be tuned too harshly I used the BitWarden random password generator to generate some random passwords and then ran it through this algorithm %ba9J$3eTvqm -> 68 (Weak) Now out of all of those passwords, it makes no sense why horse battery staple would be stronger than the other three. |
Hi, Estimating the strength of a password is not so easy. And I must admit there is no standard way to achieve this.
Nowadays, passwords with a length of 12 (~80 bytes entropy) or lower can be (statistically) guessed too easily and should be considered as risky. You can still adjust the threshold if needed. Illustration source:https://www.hivesystems.io/blog/are-your-passwords-in-the-green (also, depends on the way the passwords are stored) |
Uh oh!
There was an error while loading.Please reload this page.
Improves a bit the estimation of the entropy from#49856
/cc@Spomky