Problem/Motivation

The password recommendation does not work as expected. On other websites it is very close to the password field and not "far" under them. Also it is not clear that these rules are mandatory or recommended.

Password field problem

Proposed resolution

Password suggestion has to be clear about what kind of and how strong passwords need to be used in Drupal.

Remaining tasks

New solution has to be found for password strength suggestion.

User Interface changes

To be determined.

Api changes

Unlikely

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

davidtoltesy created an issue. See original summary.

Gábor Hojtsy’s picture

Issue summary: View changes
Gábor Hojtsy’s picture

Issue summary: View changes
Anul’s picture

FileSize
137.29 KB
137.26 KB

As initially I get all these suggestions for my password :-

To make your password stronger:

  • Make it at least 12 characters
  • Add lowercase letters
  • Add uppercase letters
  • Add numbers
  • Add punctuation

And After I start entering my password these suggestions automatically keep changing/reducing according to Drupal's password validations. Suppose I have entered "gsh@54" so i will now see only two suggestions :-

Make it at least 12 characters
Add uppercase letters

So, What else should be the changes now ? What else it requires?

And these are not mandatory, these are only recommendations.

Gábor Hojtsy’s picture

@Anul: And these are not mandatory, these are only recommendations. -- right, that is what is not clear, which this issue is about. What makes it clear from the UI that you don't need to fulfill all those rules for your password to be acceptable?

Anul’s picture

Okay Thank You @Gabor I understood and agree on the same.

What if we change some text to make it more clear that these are only recommendations and not mandatory.

Like :-
- To make your password stronger:
+ Recommendations to make your password stronger:

Make it at least 12 characters
Add lowercase letters
Add uppercase letters
Add numbers
Add punctuation

Gábor Hojtsy’s picture

@Anul that sounds like a good suggestion to me.

Anul’s picture

Status: Active » Needs review
FileSize
705 bytes

Adding a Patch for the same.

csg’s picture

Status: Needs review » Reviewed & tested by the community

This solution is very simple but it does the trick. RTBC.

Manjit.Singh’s picture

This issue was in my queue since morning ;)
Anyways +1 for RTBC. Thanks.

Gábor Hojtsy’s picture

Issue tags: +Quick fix, +Novice
webchick’s picture

Status: Reviewed & tested by the community » Fixed

Nice improvement!

Committed and pushed to 8.2.x. Because it changes strings, I did not commit to 8.1.x. Thanks!

  • webchick committed 011d2c2 on 8.2.x
    Issue #2769835 by Anul, davidtoltesy, Gábor Hojtsy, csg, Manjit.Singh:...

  • webchick committed 011d2c2 on 8.3.x
    Issue #2769835 by Anul, davidtoltesy, Gábor Hojtsy, csg, Manjit.Singh:...

  • webchick committed 011d2c2 on 8.3.x
    Issue #2769835 by Anul, davidtoltesy, Gábor Hojtsy, csg, Manjit.Singh:...

Status: Fixed » Closed (fixed)

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