Project

General

Profile

Bug #21908

prise en compte du champ slug lors d'un SSO vers FC

Added by Benjamin Dauvergne over 5 years ago. Updated almost 5 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
Benjamin Dauvergne
Target version:
future
Start date:
15 February 2018
Due date:
% Done:

100%

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

Description

commit 298c7faca75e31081403fb5978a3c4bca8cc80cf
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Mon Jan 22 11:54:28 2018 +0100

    add support for service slugs on login

Files

0001-keep-authentication-context-fixes-21908.patch (23.6 KB) 0001-keep-authentication-context-fixes-21908.patch Benjamin Dauvergne, 10 August 2018 12:43 PM
0001-keep-authentication-context-fixes-21908.patch (23.6 KB) 0001-keep-authentication-context-fixes-21908.patch Benjamin Dauvergne, 10 August 2018 12:49 PM
0002-update-french-translation-21908.patch (6.59 KB) 0002-update-french-translation-21908.patch Benjamin Dauvergne, 10 August 2018 12:49 PM
0001-keep-authentication-context-fixes-21908.patch (25.5 KB) 0001-keep-authentication-context-fixes-21908.patch Benjamin Dauvergne, 06 September 2018 11:53 AM
0002-update-french-translation-21908.patch (5.79 KB) 0002-update-french-translation-21908.patch Benjamin Dauvergne, 06 September 2018 11:53 AM
0001-keep-authentication-context-fixes-21908.patch (32.1 KB) 0001-keep-authentication-context-fixes-21908.patch Benjamin Dauvergne, 10 September 2018 04:02 PM
0002-update-french-translation-21908.patch (5.79 KB) 0002-update-french-translation-21908.patch Benjamin Dauvergne, 10 September 2018 04:02 PM
0001-keep-authentication-context-fixes-21908.patch (32.1 KB) 0001-keep-authentication-context-fixes-21908.patch Benjamin Dauvergne, 18 September 2018 05:54 PM
0002-update-french-translation-21908.patch (5.79 KB) 0002-update-french-translation-21908.patch Benjamin Dauvergne, 18 September 2018 05:54 PM
0001-views-pass-service-field-to-fc-login-post-registrati.patch (977 Bytes) 0001-views-pass-service-field-to-fc-login-post-registrati.patch Benjamin Dauvergne, 08 October 2018 12:21 PM

Associated revisions

Revision 2e5ac496 (diff)
Added by Benjamin Dauvergne about 5 years ago

keep authentication context (fixes #21908)

- simplify and reorganize login templates,
- URL are not built inside templates anymore,
- we have now 3 different templates:
- login.html for the login page
- registration.html for the registration page
- linking.html for the account page
- using feature from #25623, authentication_method is kept by the
registration view.
- the service slug is correctly threaded between every views.
- explanations about FranceConnect are now done in a common template
"explanation.html".
- restore popup mode, use it through setting A2_FC_POPUP=True, it works
for:
- login and login with registration (workflow for login with
registration is a bit complicated),
- registration,
- and linking (linking your existing to FC through the "My account"
page)
unlinking is not handled with a popup.

Revision 6429b394 (diff)
Added by Benjamin Dauvergne about 5 years ago

update french translation (#21908)

Revision 5f854490 (diff)
Added by Benjamin Dauvergne almost 5 years ago

add missing close tag (#21908)

Revision 92040ffe (diff)
Added by Benjamin Dauvergne almost 5 years ago

views: pass service field to fc login post registration (fixes #21908)

Revision 7a54118a (diff)
Added by Benjamin Dauvergne almost 5 years ago

views: pass service field to fc login post registration (fixes #21908)

Revision a041be46 (diff)
Added by Benjamin Dauvergne over 4 years ago

keep authentication context (fixes #21908)

- simplify and reorganize login templates,
- URL are not built inside templates anymore,
- we have now 3 different templates:
- login.html for the login page
- registration.html for the registration page
- linking.html for the account page
- using feature from #25623, authentication_method is kept by the
registration view.
- the service slug is correctly threaded between every views.
- explanations about FranceConnect are now done in a common template
"explanation.html".
- restore popup mode, use it through setting A2_FC_POPUP=True, it works
for:
- login and login with registration (workflow for login with
registration is a bit complicated),
- registration,
- and linking (linking your existing to FC through the "My account"
page)
unlinking is not handled with a popup.

Revision a9823e5a (diff)
Added by Benjamin Dauvergne over 4 years ago

update french translation (#21908)

Revision d3c163a6 (diff)
Added by Benjamin Dauvergne over 4 years ago

add missing close tag (#21908)

Revision 929b58d9 (diff)
Added by Benjamin Dauvergne over 4 years ago

views: pass service field to fc login post registration (fixes #21908)

History

#1

Updated by Benjamin Dauvergne over 5 years ago

  • Status changed from Solution déployée to Nouveau

Ne marche pas.

#2

Updated by Benjamin Dauvergne over 5 years ago

  • Target version changed from 0.10 to 0.11
#3

Updated by Benjamin Dauvergne over 5 years ago

  • Target version changed from 0.11 to future
#4

Updated by Benjamin Dauvergne about 5 years ago

Retirer <p>{{ login_url }}</p> de debug.

#6

Updated by Frédéric Péters about 5 years ago

(contexte pas évident à saisir)

- login.html for the login page
- registration.html for the registration page

Il y a quand même un moment où login.html est appelé mais pour une sorte d'inscription, (fc_user_info, registration_url, etc.), ça correspond à quel moment, ça ne gagnerait pas son propre template également ?

#7

Updated by Benjamin Dauvergne about 5 years ago

Frédéric Péters a écrit :

(contexte pas évident à saisir)

Je sais il y a encore du boulot de simplification, ça date du moment où Mike a injecté les fonctions d'enregistrement plus par monkeypatching que par véritable intégration entre a2 et a2-auth-fc.

- login.html for the login page
- registration.html for the registration page

Il y a quand même un moment où login.html est appelé mais pour une sorte d'inscription, (fc_user_info, registration_url, etc.), ça correspond à quel moment, ça ne gagnerait pas son propre template également ?

Ok, je vais séparer en deux, c'est effectivement quand le SSO a eu lieu mais n'a pas donné lieu à une authentification, alors on propose un bouton en page de login qui fait un enregistrement immédiat (génération d'un lien d'activation qui est transmis immédiatement).

Une fois ce commit intégré mon objectif c'est d'intégrer toute l'application dans a2, je pense que j'intégrerai le module beid par la même occasion.

#8

Updated by Mikaël Ates about 5 years ago

J'ai vu passé un credits to me alors je jette un oeil. Il y a du pdb qui traîne à la fin du test.

#9

Updated by Benjamin Dauvergne about 5 years ago

Voilà j'ai séparé login.html en 2, la partie enregistrement dans login_registration.html.

Ça dépend de #25623 pour pouvoir passe la méthode d'authentification à la vue d'enregistrememt.

#10

Updated by Frédéric Péters about 5 years ago

Le bout "popup", il peut arriver dans quelles situations ?

Taper le contenu de src/authentic2_auth_fc/templates/authentic2_auth_fc/explanation.html dans un <div class="a2-fc-explanation"> ?

Ce template ne se trouve pas utilisé dans src/authentic2_auth_fc/templates/authentic2_auth_fc/registration.html, il me semble qu'il pourrait sans soucis.

#11

Updated by Benjamin Dauvergne about 5 years ago

Voilà explanation.html est utilisé partout ou c'est utile et le fonctionnement
avec popup est de nouveau fonctionnel (pour ceux qui aimeraient).

#12

Updated by Emmanuel Cazenave about 5 years ago

Pour quoi le template explanation.html ferme une div qu'il n'a pas ouverte ?

#13

Updated by Benjamin Dauvergne about 5 years ago

Parce qu'il est méchant.

#15

Updated by Emmanuel Cazenave about 5 years ago

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

Updated by Benjamin Dauvergne almost 5 years ago

  • Status changed from Solution validée to Résolu (à déployer)
  • % Done changed from 0 to 100
#17

Updated by Benjamin Dauvergne almost 5 years ago

Et donc la div de explanation n'était pas là pour rien, j'ai du pousser un fix.

#19

Updated by Benjamin Dauvergne almost 5 years ago

J'ai oublié un bout.

#20

Updated by Benjamin Dauvergne almost 5 years ago

  • Status changed from Solution proposée to Résolu (à déployer)
#21

Updated by Benjamin Dauvergne almost 5 years ago

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

Also available in: Atom PDF