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
Fichiers
Révisions associées
update french translation (#6805)
Historique
Mis à jour par Benjamin Dauvergne il y a environ 9 ans
- Fichier 0001-report-all-password-requirements-at-once-on-password.patch 0001-report-all-password-requirements-at-once-on-password.patch ajouté
- Fichier 0002-update-french-translations-6805.patch 0002-update-french-translations-6805.patch ajouté
- Assigné à mis à Benjamin Dauvergne
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a environ 9 ans
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.
Mis à jour par Benjamin Dauvergne il y a environ 9 ans
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.
Mis à jour par Benjamin Dauvergne il y a environ 9 ans
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.
Mis à jour par Benjamin Dauvergne il y a environ 9 ans
- Fichier 0001-report-all-password-requirements-at-once-on-password.patch 0001-report-all-password-requirements-at-once-on-password.patch ajouté
- Fichier 0002-update-french-translations-6805.patch 0002-update-french-translations-6805.patch ajouté
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."
Mis à jour par Frédéric Péters il y a presque 9 ans
I'm still fine with it but the patch no longer apply.
Mis à jour par Benjamin Dauvergne il y a presque 9 ans
- Statut changé de Nouveau à Résolu (à déployer)
- % réalisé changé de 0 à 100
Appliqué par commit authentic2|00d4fac398cc73b6c69436fd9b7f1a4f090c879a.
Mis à jour par Benjamin Dauvergne il y a environ 8 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
- Statut changé de Solution déployée à Fermé
report all password requirements at once on password input (fixes #6805)