I installed this module, but I noticed I was able to create passwords with only 1 character?

On my normal password reset I had : https://www.drupal.org/project/password_policy installed, but it seems this module and password_policy don't go together?

CommentFileSizeAuthor
#5 prlp-7.x-1.x-add_password_policy_compatibility-2294381-5.patch723 bytesaohrvetpv

Comments

kyuubi’s picture

kyuubi commented
Category: Feature request » Support request

I am having the exact same issue!

Anyone ?

jitesh doshi’s picture

jitesh doshi commented

Let me look into it.

kyuubi’s picture

kyuubi commented

Hi Jitesh,

I have managed to solve this by changing password policy as the problem seems to be there.

Here is the link to the issue and the patch I provided:

https://www.drupal.org/node/2362023#comment-9273063

Anonymous’s picture

Anonymous (not verified) commented

Looks like your change in dev either wasn't working or was not accepted kyuubi. At this time the Dev version of password policy actually makes things worse. At least when the 1 time email link is used with the prod version of pass policy it will show you what requirements (alpha, at least 1 symbol, >7 etc) are needed. But the dev version now (when using 1 time email link) doesn't even show that, it just shows the standard drupal message of make it at least 6, use upper / lower case etc.

aohrvetpv’s picture

aohrvetpv commented
Status: Active » Needs review
StatusFileSize
new prlp-7.x-1.x-add_password_policy_compatibility-2294381-5.patch723 bytes

This patch adds compatibility with the latest version of Password Policy 7.x-2.x. The solution is inelegant though; it works by modifying hook execution order to ensure Password Policy calls form_alter hooks after PRLP. Perhaps there exists a simpler solution.

How Password Policy modifies the password element has been changed, so the patch referenced in #3 no longer works. Description of the problem here: #2362023-27: Support for Password Reset Landing Page (PRLP).

Anonymous’s picture

Anonymous (not verified) commented

The patch does solve the problem, which is critical. Thank you.

jitesh doshi’s picture

jitesh doshi commented
Status: Needs review » Fixed

Patch applied to 7.x-1.x branch. Please test.

Status: Fixed » Closed (fixed)

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

a_thakur’s picture

a_thakur commented

I see this can be committed..could we have another release as well for this module?

Vinnie D’s picture

Vinnie D commented

New release created: 7.x-1.2, which includes the fix for said issue.

a_thakur’s picture

a_thakur commented

Thanks @Vinnie D