Works by adjusting module weights, which is ugly. Probably better to use after_update as suggested in #598424-26: Secure Password Hashes (phpass) and Password module integration.

CommentFileSizeAuthor
#2 password_policy-improve_phpass_support-2416407-2.patch1.22 KBaohrvetpv
#1 password_policy-improve_phpass_support-2416407-1.patch1.32 KBaohrvetpv

Comments

aohrvetpv’s picture

aohrvetpv commented
Status: Active » Needs review
StatusFileSize
new password_policy-improve_phpass_support-2416407-1.patch1.32 KB

Probably needs some refinement, but all tests pass both with and without Secure Password Hashes.

I wonder if it would be better to not change the module weight back to 0 if it is not 1 as expected. An administrator may have set a custom weight for the module.

aohrvetpv’s picture

aohrvetpv commented
StatusFileSize
new password_policy-improve_phpass_support-2416407-2.patch1.22 KB

Previous patch had an incorrect condition.

Mystery: Testbot indicates 20 more passes than when I run tests locally.

AohRveTPV queued 2: password_policy-improve_phpass_support-2416407-2.patch for re-testing.

  • AohRveTPV committed b24727b on 6.x-1.x 
    Issue #2416407 by AohRveTPV: Improve Secure Password Hashes integration
aohrvetpv’s picture

aohrvetpv commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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