Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
I filled out password constraint form.. chose 3 uppercase and 2 lowercase characters. What's not obvious in the UI when adding a password is that the 3 uppercase characters can't be the same character. Policy status message says "Fail - Password must contain at 3 uppercase characters" and "Fail - Password must contain at least 2 lowercase characters". Seems like the status message should contain the word distinct or unique to be more clear.
Comment | File | Size | Author |
---|---|---|---|
#11 | password_policy_2894929.png | 69.46 KB | jessicacs |
#7 | password_policy-updated_message.png | 86.58 KB | baikho |
#6 | interdiff_1-6.txt | 4.13 KB | baikho |
#6 | password_policy-password_character_plugin_corrected_message-2894929-6.patch | 5.59 KB | baikho |
| |||
#2 | issue-2894929-change_message-1.patch | 1.58 KB | jayvolk |
Issue fork password_policy-2894929
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #2
jayvolk CreditAttribution: jayvolk as a volunteer and commentedpatch for review
Comment #3
daggerhart CreditAttribution: daggerhart commentedRe-queued the test and everything passed. The code here looks good and the text in the patch provides more clarity for the policy.
Comment #4
AohRveTPV CreditAttribution: AohRveTPV commentedShould also be clarified for numeric and special characters.
Thanks for the issue and patch.
Comment #5
AohRveTPV CreditAttribution: AohRveTPV commentedI think this could be considered a bug.
Comment #6
baikhoSame here, 1 of our clients has also highlighted this issue.
See attached patch with addressed change on comment #4 including plugin summary and the test updates.
Comment #7
baikhoComment #10
hmendes CreditAttribution: hmendes at CI&T commentedRe-rolling patch from #6 as a MR.
Comment #11
jessicacs CreditAttribution: jessicacs at CI&T commentedI tested issue from MR and it worked for me.
Comment #14
paulocsThanks!