Skip to content
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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Password strength meter shows incorrect result #10122

Open
notramo opened this issue Jun 20, 2019 · 4 comments
Open

Password strength meter shows incorrect result #10122

notramo opened this issue Jun 20, 2019 · 4 comments
Labels
A-Registration O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect Z-Upstream

Comments

@notramo
Copy link

notramo commented Jun 20, 2019

asdf1fdsa1 is NOT a strong password. Even a human can guess it. Riot web says it is a good password.
The password strength meter should be stricter. E.g. at least 15 characters should be considered medium strength, and 20 should be strong, but only if uppercase letters, numbers, and non-word characters are used. If it consists of lowercase letters and numbers, it should be even longer.
@akvadrako akvadrako mentioned this issue Aug 24, 2020
Open
@t3chguy
Copy link
Member

t3chguy commented Aug 24, 2020

https://lowe.github.io/tryzxcvbn/ gives asdf1fdsa1 3/4 score, I suggest you open this issue upstream https://github.com/dropbox/zxcvbn as setting the requirement to 4/4 would require too strong passwords.

@notramo notramo mentioned this issue Aug 24, 2020
Open
@jryans jryans removed the Z-UI/UX label Mar 9, 2021
@ritikBhandari
Copy link

ritikBhandari commented Mar 25, 2022 •
edited

Hi,
Can I try resolve this issue if still open?

@SimonBrandner
Copy link
Contributor

Hi, Can I try resolve this issue if still open?

From the looks of it, dropbox/zxcvbn#292 hasn't been resolved yet, so I am not sure how would you like to approach fixing the issue?

@SimonBrandner SimonBrandner added S-Minor Impairs non-critical functionality or suitable workarounds exist O-Occasional Affects or can be seen by some users regularly or most users rarely A-Registration labels Mar 27, 2022
@ritikBhandari
Copy link

Ohh is it? I didn't know about the previous issue. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Registration O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect Z-Upstream
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@jryans @t3chguy @SimonBrandner @notramo @ritikBhandari