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.

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

Repeated patterns accepted too easily? #1

Open
timds opened this issue Dec 9, 2019 · 3 comments
Open

Repeated patterns accepted too easily? #1

timds opened this issue Dec 9, 2019 · 3 comments

Comments

@timds
Copy link

timds commented Dec 9, 2019

Hi,

In dropbox's zxcvbn library, which I can test here, passwords with clearly repeating patterns get scored lowly. E.g. "ab ab ab" and "aa aaa aa" get scored as 1/4. In zxcvbn-hs, the same passwords are considered "Safe".

Is there a bug in the pattern checking?

(Aside: By contrast, the dictionary checking seems to be stricter: e.g. "set your password" is considered "Moderate" by zxcvbn-hs but 4/4 by dropbox's library)

@pjones
Copy link
Collaborator

pjones commented Dec 9, 2019

Hi @timds.

zxcvbn-hs is not an exact line-by-line port of the dropbox library. I based it on the USENIX paper and used the dropbox implementation as a guide when the paper was vague. In other words, they aren't always going to agree with one another.

The pattern checking feature is very different in my implementation. In the original version the password fragments go through a regular expression engine multiple times to find the patterns. In the Haskell version no regular expressions are used.

If you feel there are specific checkers that are providing erroneous scores we should talk about that. However, based on the password cracking techniques that I'm familiar with, checkers like the pattern finder aren't that valuable in practice.

@timds
Copy link
Author

timds commented Dec 10, 2019

That sounds reasonable to me, thanks. Still, intuitively, "ab ab ab" at 8 characters doesn't seem particularly safe, and I wonder whether a short password with repetitions like that could be penalised slightly more than it is currently, but I agree it may not be worth it if it's not a trivial change.

@pjones
Copy link
Collaborator

pjones commented Dec 10, 2019

I'm going to leave this issue open as a reminder to look at the pattern code and see if there is a way to make something like this have a lower score.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants