-
-
Notifications
You must be signed in to change notification settings - Fork 26
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
Not always using all the types in the pattern #31
Comments
I know this has been a while, but I'm getting this updated and wanted to address this. The readme docs indicate that any of those characters may be used. I didn't see anything that said that all the characters would be used though. However, I do think it should provide that option. I'll leave this open for commentary while I work on updating the lib. |
Due to this bug, we are unable to use this in a system that automatically "tests" the strength of the password based on it containing a specific property (e.g checking for uppercase, lowercase, numeric, and special characters). I appreciate the irony that attempting to enforce entropy in some respects decreases it, but estimating password strength in our use case trumps this. |
const password = randomize("aA0", 8); console.log({password}); //{ password: 'lfwWBUif' }
in this case it didn't include any Numeric character
The text was updated successfully, but these errors were encountered: