Skip to content

fix: README #44

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

Open
davegudge wants to merge 1 commit into
base: master
Choose a base branch
from
+1 −1
Open

fix: README #44

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -137,7 +137,7 @@ between 3 and 6 in length and reduces the entropy of the password accordingly.
Finally, if specified, we will run the password through a `DictionaryAdjuster`. This is the slowest part of the
algorithm but adds significantly to the strength checking. Ideally this should be run against a large dictionary
of the most commong words in your language (e.g. a 300,000 word English dictionary). Currently I just have this
hard-coded to run against an embedded list of the 500 most common passwords. This is much faster than running
hard-coded to run against an embedded list of the [9,995](https://github.com/bdmac/strong_password/blob/6ec0701986dcf240533a29d9fcae52c9e0515b99/lib/strong_password/dictionary_adjuster.rb#L3) most common passwords. This is much faster than running
against a 300,000 dictionary file and still adds considerably to the strength checking. The password is checked
against all dictionary words over a minimum word length (default is 4). We also generate variations of
the password to check for common things like leet speak, and qwerty-shifted passwords. The lowest entropy found
Expand Down