-
Notifications
You must be signed in to change notification settings - Fork 2
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
Platform: Better Passwords #272
Comments
This is up and available for review at https://pr-1189-wriflagship.pantheonsite.io/admin/people/create If you try to enter a password that's less than 12 characters, you will get an error. Weak passwords like "testtesttest" also throw errors. You also have the option of auto-generating a password for a new user. This makes a random string of characters for you. In this workflow, admins never know the generated password. New users must use the "Forgot password" link, check their emails, and generate a new password for themselves to log in for the first time. Using the auto-generated password is optional though. |
Note: Password reset needs SendGrid enabled on platform sites |
Change is up on develop: The auto-generate option is now available on Flagship since it has sendgrid set up: |
Hmm, I did get an email sent from sendgrid: @shannon-paton There's a checkbox below that is easy to miss but is required to get the email to send "Notify user of new account" -- any chance you missed checking that? |
Yep, that was it! Worked for me @mariacha |
I've tested this out on the IOs and will deploy tomorrow. |
Deployed today, so closing! |
Assess a password tool like https://www.drupal.org/project/better_passwords. Requires the zxcvbn-php library available at https://github.com/bjeavons/zxcvbn-php but otherwise seems straightforward.
The text was updated successfully, but these errors were encountered: