Strength and DRY password form fields #41

Closed
wants to merge 2 commits into
from

Projects

None yet

2 participants

@gustavodiazjaimes

The changes provide the ability to enable secure passwords
settings.ACCOUNT_STRENGTH_PASS = True
The used algorithm is from a Jquery plugin:
http://phiras.wordpress.com/2007/04/08/password-strength-meter-a-jquery-plugin/

Help request:

  • Must write a better error message and update the i18n
  • Assembling the Jquery plugin in the template
@AliLozano

What's happened with this commit?.. there are other way to validate a strength password?.

@gustavodiazjaimes

The commit works, I didn't find other way, just realize there is not a feature pinax need.

@AliLozano

How can i use this?.. I don't see this changes in a master branch, and can't find a branch with this changes to install from pip.

@gustavodiazjaimes

The change was only proposed here and is only a fork of the 2 files u can't see in the commit diff, so:

  • The files still haven't change so you can simple replace them in a clean pinax instalation or deal with diff
  • Declare a variable ACCOUNT_STRENGTH_PASS in u'r project settings.py ** if ACCOUNT_STRENGTH_PASS is True, strengh = 34 ** if ACCOUNT_STRENGTH_PASS is Number, strengh = ACCOUNT_STRENGTH_PASS

look at http://phiras.wordpress.com/2007/04/08/password-strength-meter-a-jquery-plugin/ to estimated strengh

@AliLozano

Thanks, do you know why they don't merge this with the master branch?

@gustavodiazjaimes

I guess I close it to early :P

Me alegra que te haya servido

@AliLozano

Pues si, aunque me hubiese gustado que el cambio hubiese sido agregado a la rama master .
Gracias por todo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment