Project

General

Profile

Development #27083

Faire disparaître le lien de changement de mot de passe quand on est connecté via FC

Added by Benjamin Dauvergne 8 months ago. Updated about 1 month ago.

Status:
Fermé
Priority:
Normal
Target version:
-
Start date:
08 Oct 2018
Due date:
% Done:

0%

Hors marché:
No
Patch proposed:
Yes
Planning:
No

Description

Pour éviter que les gens soient perdus entre leur mot de passe FranceConnect et celui dans Publik.

0001-apps-hide-password-management-link-when-FC-authentic.patch View (6.39 KB) Benjamin Dauvergne, 03 May 2019 10:14 AM

0002-tests-use-pg_virtualenv-with-on-jenkins-27083.patch View (1.65 KB) Benjamin Dauvergne, 03 May 2019 04:05 PM

0001-apps-hide-password-management-link-when-FC-authentic.patch View (6.22 KB) Benjamin Dauvergne, 03 May 2019 04:05 PM

0002-tests-use-pg_virtualenv-with-on-jenkins-27083.patch View (1.65 KB) Benjamin Dauvergne, 03 May 2019 04:48 PM

0001-apps-hide-password-management-link-when-FC-authentic.patch View (6.12 KB) Benjamin Dauvergne, 03 May 2019 04:48 PM


Related issues

Related to Publik - Development #27081: Intégration cahier des charges FranceConnect Nouveau 08 Oct 2018
Related to Authentic 2 - Development #28848: Avoir un hook pour autoriser le changement de mot de passe Solution déployée 11 Dec 2018
Related to Authentic 2 - Development #32780: utils: ajouter une méthode get_authentication_events_hows Solution déployée 03 May 2019
Related to Plugin FS FranceConnect - Development #32953: quand on délie son compte de FC le lien de changement de mot de passe devrait ré-apparaître Solution déployée 09 May 2019

Associated revisions

Revision ce56e15d (diff)
Added by Benjamin Dauvergne about 1 month ago

apps: hide password management link when FC authenticated (#27083)

It's done by implementing a2_hook_user_can_change_password on the
AppConfig object.

Revision 7f191e9b (diff)
Added by Benjamin Dauvergne about 1 month ago

tests: use pg_virtualenv with on jenkins (#27083)

Revision 3ca5c43a (diff)
Added by Benjamin Dauvergne 16 days ago

apps: hide password management link when FC authenticated (#27083)

It's done by implementing a2_hook_user_can_change_password on the
AppConfig object.

History

#1 Updated by Benjamin Dauvergne 8 months ago

#2 Updated by Benjamin Dauvergne 6 months ago

  • Related to Development #28848: Avoir un hook pour autoriser le changement de mot de passe added

#3 Updated by Benjamin Dauvergne about 1 month ago

  • Assignee set to Benjamin Dauvergne

#4 Updated by Benjamin Dauvergne about 1 month ago

Nécessite #32760 pour que les tests passent.

#5 Updated by Thomas Noël about 1 month ago

Je trouve que la partie qui cherche dans les events le "how" égal à "france-connect" mériterait plutôt une fonction dans authentic/utils.py ; du genre find_authentication_event_hows qui reverrait la liste des how, et on ferait alors juste « return "france-connect" in authentication_event_hows(requests) » ?

(juste pour éviter d'avoir à se souvenir que si on change un jour la gestion des authentication_events dans authentic, il faut le faire aussi dans les plugin)

Je sais pas ce que tu en dis, Benjamin...

#6 Updated by Benjamin Dauvergne about 1 month ago

  • Related to Development #32780: utils: ajouter une méthode get_authentication_events_hows added

#7 Updated by Benjamin Dauvergne about 1 month ago

Voilà #32780

#9 Updated by Thomas Noël about 1 month ago

  • Status changed from Solution proposée to Solution validée

Benjamin Dauvergne a écrit :

En utilisant le nouveau get_authentication_events()

Tu peux simplifier un poil si tu veux :

        if request.session:
            authentication_events = get_authentication_events(request=request)
            for authentication_event in authentication_events:

peut devenir juste :

       for authentication_event in get_authentication_events(request=request):

ack avec ou sans ça.

#10 Updated by Benjamin Dauvergne about 1 month ago

Yes.

#11 Updated by Thomas Noël about 1 month ago

(Il faut pousser #32780 pour voir le résultat des tests)

#12 Updated by Thomas Noël about 1 month ago

  • Status changed from Solution proposée to Solution validée

Et ack si les tests passent.

#13 Updated by Benjamin Dauvergne about 1 month ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 7f191e9bc6a0e3fc577ad903a3d02a08c9d3bca3
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Fri May 3 11:54:14 2019 +0200

    tests: use pg_virtualenv with on jenkins (#27083)

commit ce56e15d0bc98d480f456bc8a7b633327160608d
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Fri May 3 10:11:47 2019 +0200

    apps: hide password management link when FC authenticated (#27083)

    It's done by implementing a2_hook_user_can_change_password on the
    AppConfig object.

#14 Updated by Benjamin Dauvergne about 1 month ago

  • Related to Development #32953: quand on délie son compte de FC le lien de changement de mot de passe devrait ré-apparaître added

#15 Updated by Benjamin Dauvergne about 1 month ago

  • Status changed from Résolu (à déployer) to Fermé

Also available in: Atom PDF