Development #69989
Section "Changement de mot de passe" - NOK (point 26)
0%
Description
Un 26ème point ajouté, le 29/09, par la DINUM à la 2ème passe de la recette "Nouvelle certification".
26. Section "Changement de mot de passe" - NOK https://connexion-angouleme.test.entrouvert.org/accounts/password/change/ Je rajoute ce point, car je pensais que cette section ne s'affichait pas en cas de connexion via FranceConnect, mais en fait si (voir screen ci-dessous). Il faut donc indiquer qu'il s'agit du mot de passe du compte local, qu’il n’y a pas de lien avec FranceConnect. Merci donc d'ajouter la phrase suivante sous le titre : « Attention, ce mot de passe est celui de votre compte local et en aucun cas celui du compte que vous utilisez au travers de FranceConnect. Il vous servira uniquement lorsque vous vous connecterez avec votre adresse mail plutôt que via FranceConnect. »Déroulé :
- "s'identifier FC" avec test/123 sur https://connexion-angouleme.test.entrouvert.org/login/
- dans https://connexion-angouleme.test.entrouvert.org/accounts/ "Gestion du compte" : 3 lignes et "Modifier le mot de passe" n'apparaît pas : OK
- "Supprimer la liaison", choisir un mot de passe local, "Délier"
- retour sur https://connexion-angouleme.test.entrouvert.org/accounts/ "Gestion du compte" : 4 lignes dont "Modifier le mot de passe" : OK
- "Lier votre compte utilisateur à FranceConnect." avec clic sur bouton FC, s'identifier dans FC avec test/123
- retour sur https://connexion-angouleme.test.entrouvert.org/accounts/ "Gestion du compte" : 4 lignes dont "Modifier le mot de passe"
=> et là DINUM considère que problème et qu'il faudrait donc ajouter : " Attention, ce mot de passe est celui de votre compte local et en aucun cas celui du compte que vous utilisez au travers de FranceConnect. Il vous servira uniquement lorsque vous vous connecterez avec votre adresse mail plutôt que via FranceConnect. "
Files
Related issues
Associated revisions
auth_fc: show warning on password change page if user is linked to FranceConnect (#69989)
translation update (#69989)
History
Updated by Brice Mallet 4 months ago
- Related to Development #68360: Cinématique de Réconciliation de compte - NOK added
Updated by Brice Mallet 4 months ago
- Related to Project management #68358: Demandes d'évolution pour la nouvelle certification FranceConnect added
Updated by Benjamin Dauvergne 2 months ago
- File 0001-auth_fc-store-id_token-in-session-when-linking-69989.patch 0001-auth_fc-store-id_token-in-session-when-linking-69989.patch added
- File 0002-auth_fc-show-warning-on-password-change-page-if-user.patch 0002-auth_fc-show-warning-on-password-change-page-if-user.patch added
- Tracker changed from Project management to Development
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
- la vue de mot de passe ne sera plus visible dans le cas particulier où on s'est connecté avec FranceConnect, puis délier, puis relier.
- si jamais le compte courant est relié à FranceConnect mais que la vue de changement de mot de passe est visible (ex.: on s'est connecté via son email/mot de passe et pas par FranceConnect) alors on affiche un disclaimer.
Updated by Benjamin Dauvergne 2 months ago
- File 0001-auth_fc-store-id_token-in-session-when-linking-69989.patch 0001-auth_fc-store-id_token-in-session-when-linking-69989.patch added
- File 0003-translation-update-69989.patch 0003-translation-update-69989.patch added
- File 0002-auth_fc-show-warning-on-password-change-page-if-user.patch 0002-auth_fc-show-warning-on-password-change-page-if-user.patch added
Avec la trad.
Updated by Serghei Mihai 2 months ago
def get(self, request, *args, **kwargs):
return super().get(request, *args, **kwargs)
n'est pas inutile ?
Updated by Benjamin Dauvergne 2 months ago
- File 0001-auth_fc-store-id_token-in-session-when-linking-69989.patch 0001-auth_fc-store-id_token-in-session-when-linking-69989.patch added
- File 0003-translation-update-69989.patch 0003-translation-update-69989.patch added
- File 0002-auth_fc-show-warning-on-password-change-page-if-user.patch 0002-auth_fc-show-warning-on-password-change-page-if-user.patch added
Updated by Paul Marillonnet 2 months ago
Dans 0001 peut-être moyen de factoriser du code avec ce qui est fait dans le cas où il y a login et non pas liaison dans le code à la fin handle_authorization_response
if request.user.is_authenticated:
return self.link(request)
else:
return self.login(request)
Par exemple dans le cas login je vois qu’en plus il y a paramétrage de l’expiration de la session, après stockage de l’ID token :
# set session expiration policy to EXPIRE_AT_BROWSER_CLOSE
request.session.set_expiry(0)
0002 ok, peut-être juste
· s/not the one from the account used through FranceConnect/not the one from your FranceConnect provider/
· s/connect with locally/log in locally/
dans le message affiché à l’usager ?
Updated by Benjamin Dauvergne 2 months ago
Paul Marillonnet a écrit :
Dans 0001 peut-être moyen de factoriser du code avec ce qui est fait dans le cas où il y a login et non pas liaison dans le code à la fin
handle_authorization_response
[...] ?
Par exemple dans le cas login je vois qu’en plus il y a paramétrage de l’expiration de la session, après stockage de l’ID token :
[...] (dans finish_login) qui n’est pas repris dans 0001 ?
Ça ne concerne qu'une session ouverte par FranceConnect, la session ayant pu être ouverte via mot de passe, je préfère ne pas changer (#40479).
0002 ok, peut-être juste
· s/not the one from the account used through FranceConnect/not the one from your FranceConnect provider/
· s/connect with locally/log in locally/
dans le message affiché à l’usager ?
Ok.
Updated by Benjamin Dauvergne 2 months ago
- File 0001-auth_fc-store-id_token-in-session-when-linking-69989.patch 0001-auth_fc-store-id_token-in-session-when-linking-69989.patch added
- File 0003-translation-update-69989.patch 0003-translation-update-69989.patch added
- File 0002-auth_fc-show-warning-on-password-change-page-if-user.patch 0002-auth_fc-show-warning-on-password-change-page-if-user.patch added
Voilà.
Updated by Paul Marillonnet 2 months ago
- Status changed from Solution proposée to Solution validée
Benjamin Dauvergne a écrit :
Ça ne concerne qu'une session ouverte par FranceConnect, la session ayant pu être ouverte via mot de passe, je préfère ne pas changer (#40479).
Ok.
Ok.
Ok, peut-être juste un s/mail/courriel/ dans le message traduit dans 0003 puisque cela semble être la convention dans authentic.
Ack.
Updated by Benjamin Dauvergne 2 months ago
- File 0001-auth_fc-store-id_token-in-session-when-linking-69989.patch 0001-auth_fc-store-id_token-in-session-when-linking-69989.patch added
- File 0003-translation-update-69989.patch 0003-translation-update-69989.patch added
- File 0002-auth_fc-show-warning-on-password-change-page-if-user.patch 0002-auth_fc-show-warning-on-password-change-page-if-user.patch added
- Status changed from Solution validée to Solution proposée
Oublié de pousser le test.
Updated by Paul Marillonnet 2 months ago
- Status changed from Solution proposée to Solution validée
Benjamin Dauvergne a écrit :
Oublié de pousser le test.
Magnifique. Reste juste pylint à contenter :
tests/auth_fc/test_views.py:25: [C1802(use-implicit-booleaness-not-len), test_password_change_view_with_fc] Do not use `len(SEQUENCE)` without comparison to determine if a sequence is empty
et c’est bon.
Updated by Benjamin Dauvergne 2 months ago
- Status changed from Solution validée to Résolu (à déployer)
commit a8ebccc5aa8ec83f9db0e0edee8b309be40def28
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Tue Nov 22 15:37:37 2022 +0100
translation update (#69989)
commit 9171c30febcb58c6af4a2514ebfbcda64d8b33db
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Tue Nov 22 15:34:26 2022 +0100
auth_fc: show warning on password change page if user is linked to FranceConnect (#69989)
commit 89b526066d6373e741c4843c6c93632febc8d2cb
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Tue Nov 22 15:08:07 2022 +0100
auth_fc: store id_token in session when linking (#69989)
Password change view is hidden only if the current session was opened
with FranceConnect and if the id_token is visible in the session, this
change will enforce the second condition when during the same session
the account is unlinked and linked again to FranceConnect.
It will also permit to launch a logout if the link fails.Updated by Transition automatique about 1 month ago
- Status changed from Résolu (à déployer) to Solution déployée
auth_fc: store id_token in session when linking (#69989)
Password change view is hidden only if the current session was opened
with FranceConnect and if the id_token is visible in the session, this
change will enforce the second condition when during the same session
the account is unlinked and linked again to FranceConnect.
It will also permit to launch a logout if the link fails.