Bug #21908
prise en compte du champ slug lors d'un SSO vers FC
100%
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
Fichiers
Révisions associées
update french translation (#21908)
add missing close tag (#21908)
views: pass service field to fc login post registration (fixes #21908)
views: pass service field to fc login post registration (fixes #21908)
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.
update french translation (#21908)
add missing close tag (#21908)
views: pass service field to fc login post registration (fixes #21908)
Historique
Mis à jour par Benjamin Dauvergne il y a environ 6 ans
- Statut changé de Solution déployée à Nouveau
Ne marche pas.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Fichier 0001-keep-authentication-context-fixes-21908.patch 0001-keep-authentication-context-fixes-21908.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Retirer <p>{{ login_url }}</p>
de debug.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Fichier 0001-keep-authentication-context-fixes-21908.patch 0001-keep-authentication-context-fixes-21908.patch ajouté
- Fichier 0002-update-french-translation-21908.patch 0002-update-french-translation-21908.patch ajouté
avec mise à jour des traductions.
Mis à jour par Frédéric Péters il y a plus de 5 ans
(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 ?
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
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 pageIl 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.
Mis à jour par Mikaël Ates il y a plus de 5 ans
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.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Fichier 0001-keep-authentication-context-fixes-21908.patch 0001-keep-authentication-context-fixes-21908.patch ajouté
- Fichier 0002-update-french-translation-21908.patch 0002-update-french-translation-21908.patch ajouté
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.
Mis à jour par Frédéric Péters il y a plus de 5 ans
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.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Fichier 0001-keep-authentication-context-fixes-21908.patch 0001-keep-authentication-context-fixes-21908.patch ajouté
- Fichier 0002-update-french-translation-21908.patch 0002-update-french-translation-21908.patch ajouté
Voilà explanation.html est utilisé partout ou c'est utile et le fonctionnement
avec popup est de nouveau fonctionnel (pour ceux qui aimeraient).
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
Pour quoi le template explanation.html ferme une div qu'il n'a pas ouverte ?
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Fichier 0001-keep-authentication-context-fixes-21908.patch 0001-keep-authentication-context-fixes-21908.patch ajouté
- Fichier 0002-update-french-translation-21908.patch 0002-update-french-translation-21908.patch ajouté
explanation.html corrigé.
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
- % réalisé changé de 0 à 100
Appliqué par commit authentic2-auth-fc|2e5ac496b84b185ede3e146d0a01207837c2d162.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
Et donc la div de explanation n'était pas là pour rien, j'ai du pousser un fix.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Fichier 0001-views-pass-service-field-to-fc-login-post-registrati.patch 0001-views-pass-service-field-to-fc-login-post-registrati.patch ajouté
- Statut changé de Résolu (à déployer) à Solution proposée
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Solution proposée à Résolu (à déployer)
Appliqué par commit authentic2-auth-fc|92040ffe53bbd2b4ba00eb855a5591d8465aedb6.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Fermé
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.