-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Short Faker::Internet.password are not including mix case chars. #2512
Comments
adding a bit more context:
so maybe a regression? adding a test/reproduction scripts that can reproduce this issue would be a nice addition. |
it looks like this occurs when both |
@thdaraujo and I were able to reproduce the error with the following script:
#2308 did not solve this issue. At the moment, the following scenarios are possible:
What we want is make sure that we have at least one upper case AND one lower case, and the special characters. Examples:
|
Hi @stefannibrasil - so sorry for the delay. Yes, I was able to reproduce the issue and found #2308 did not resolve. I haven't the chance to revisit this just yet but can get my PR up in the next 24 hours. |
hi @tiff-o no worries! Take your time. I was just trying to reproduce the issue and share what we found. Thanks for working on this 👍 |
thanks @stefannibrasil! no problem - am currently working on trying to simplify |
Describe the bug
Short passwords generated with
Faker::Internet.password
does not respect the configs related to "mix_case".To Reproduce
Generate several short passwords with mixed case and special characters. Eventually, a invalid password without mixed cased chars will be generated.
Example:
Expected behavior
Passwords to have mixed case characters.
Additional context
Version:
The text was updated successfully, but these errors were encountered: