It would be awesome to be able to disallow certain passwords, for example "password".

For example, 91% of users have a password from the top1000 passwords:
http://jajodia-saket.sjbn.co/2011/08/worlds-most-used-passwords-and-some...

Being able to copy and paste a list like this into the "disallowed passwords" would prevent
the "rainbow table attacks".

Comments

erikwebb’s picture

Status: Active » Closed (won't fix)

A decent password policy should inherently prevent these types of weak passwords. If a basic policy of numbers and letters is enabled, you could allow bad passwords like "abc123", but this seems like a feature just filling in the gap for already too lenient policies.

jpklein’s picture

Version: 7.x-1.0-beta1 » 7.x-1.2
StatusFileSize
new3.69 KB


While I realize that this issue is officially marked as won't fix, I had a requirement to prohibit users from including the sitemname in their passwords, and figured this was a good place to put the patch.

Applying the patch to version 7.x-1.2 adds a simple "word list" constraint to the policy-edit form, allowing you to add a comma-separated list of strings that can not appear in users' passwords.

Note that this is not designed to handle long lists of words or protect against dictionary attacks; see #1780802: Implement a dictionary-based password constraint for development on that front.

erikwebb’s picture

Title: Don't allow certain passwords » Add constraint to blacklist certain passwords
Version: 7.x-1.2 » 7.x-2.x-dev

This should also be solved by the pluggable constraints model for 7.x-2.x. I actually think this would be a great plugin to include, if for no other reason than to write content-based automated tests.

erikwebb’s picture

Status: Closed (won't fix) » Active
aohrvetpv’s picture

Issue summary: View changes
Status: Active » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.