Bug #6805
Give all password requirements at once
100%
Description
It is frustrating to enter a password that is too short, receive a message that it must be at least n characters long, type a new one, and get another refusal because it doesn't contain the appropriate classes of characters. Password requirements should all be displayed at once.
Related upstream django discussion: https://groups.google.com/forum/#!topic/django-developers/9GBhgGXmEKs
Files
Associated revisions
update french translation (#6805)
History
Updated by Benjamin Dauvergne about 10 years ago
- File 0001-report-all-password-requirements-at-once-on-password.patch 0001-report-all-password-requirements-at-once-on-password.patch added
- File 0002-update-french-translations-6805.patch 0002-update-french-translations-6805.patch added
- Assignee set to Benjamin Dauvergne
- Patch proposed changed from No to Yes
Updated by Frédéric Péters about 10 years ago
Perhaps "Your password must match the regular expression: %(regexp)s'" will prevent people from even trying to enter a password, I would leave that one out. I would also maybe s/Your password/The password/ in the message.
Updated by Benjamin Dauvergne about 10 years ago
The regexp message is here to recall people that they must set custom message through app_settings.A2_PASSWORD_POLICY_REGEX_ERROR_MSG, it is not meant for real user consumption. If you prefer I could just add a check that the two settings are defined using the new setting checks from Django 1.7 and crashing on launch.
Updated by Benjamin Dauvergne about 10 years ago
Benjamin Dauvergne a écrit :
The regexp message is here to recall people that they must set custom message through
app_settings.A2_PASSWORD_POLICY_REGEX_ERROR_MSG, it is not meant for real user consumption. If you prefer I could just add a check that the two settings are defined using the new setting checks from Django 1.7 and crashing on launch.
I reminded me that it will not work with a multitenant deployment as I cannot check tenant settings.
Updated by Benjamin Dauvergne about 10 years ago
- File 0001-report-all-password-requirements-at-once-on-password.patch 0001-report-all-password-requirements-at-once-on-password.patch added
- File 0002-update-french-translations-6805.patch 0002-update-french-translations-6805.patch added
Message changed for :
+"Your password must match the regular expression: %(regexp)s, please change " +"this message using the A2_PASSWORD_POLICY_REGEX_ERROR_MSG setting."
Updated by Frédéric Péters almost 10 years ago
I'm still fine with it but the patch no longer apply.
Updated by Benjamin Dauvergne almost 10 years ago
- Status changed from Nouveau to Résolu (à déployer)
- % Done changed from 0 to 100
Appliqué par commit authentic2|00d4fac398cc73b6c69436fd9b7f1a4f090c879a.
Updated by Benjamin Dauvergne about 9 years ago
- Status changed from Résolu (à déployer) to Solution déployée
report all password requirements at once on password input (fixes #6805)