Projet

Général

Profil

Bug #21908

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

Ajouté par Benjamin Dauvergne il y a environ 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
Début:
15 février 2018
Echéance:
% réalisé:

100%

Temps estimé:
Hors marché:
Non
Patch proposed:
Oui
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


Fichiers

0001-keep-authentication-context-fixes-21908.patch (23,6 ko) 0001-keep-authentication-context-fixes-21908.patch Benjamin Dauvergne, 10 août 2018 12:43
0001-keep-authentication-context-fixes-21908.patch (23,6 ko) 0001-keep-authentication-context-fixes-21908.patch Benjamin Dauvergne, 10 août 2018 12:49
0002-update-french-translation-21908.patch (6,59 ko) 0002-update-french-translation-21908.patch Benjamin Dauvergne, 10 août 2018 12:49
0001-keep-authentication-context-fixes-21908.patch (25,5 ko) 0001-keep-authentication-context-fixes-21908.patch Benjamin Dauvergne, 06 septembre 2018 11:53
0002-update-french-translation-21908.patch (5,79 ko) 0002-update-french-translation-21908.patch Benjamin Dauvergne, 06 septembre 2018 11:53
0001-keep-authentication-context-fixes-21908.patch (32,1 ko) 0001-keep-authentication-context-fixes-21908.patch Benjamin Dauvergne, 10 septembre 2018 16:02
0002-update-french-translation-21908.patch (5,79 ko) 0002-update-french-translation-21908.patch Benjamin Dauvergne, 10 septembre 2018 16:02
0001-keep-authentication-context-fixes-21908.patch (32,1 ko) 0001-keep-authentication-context-fixes-21908.patch Benjamin Dauvergne, 18 septembre 2018 17:54
0002-update-french-translation-21908.patch (5,79 ko) 0002-update-french-translation-21908.patch Benjamin Dauvergne, 18 septembre 2018 17:54
0001-views-pass-service-field-to-fc-login-post-registrati.patch (977 octets) 0001-views-pass-service-field-to-fc-login-post-registrati.patch Benjamin Dauvergne, 08 octobre 2018 12:21

Révisions associées

Révision 2e5ac496 (diff)
Ajouté par Benjamin Dauvergne il y a plus de 5 ans

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.

Révision 6429b394 (diff)
Ajouté par Benjamin Dauvergne il y a plus de 5 ans

update french translation (#21908)

Révision 5f854490 (diff)
Ajouté par Benjamin Dauvergne il y a plus de 5 ans

add missing close tag (#21908)

Révision 92040ffe (diff)
Ajouté par Benjamin Dauvergne il y a plus de 5 ans

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

Révision 7a54118a (diff)
Ajouté par Benjamin Dauvergne il y a plus de 5 ans

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

Révision a041be46 (diff)
Ajouté par Benjamin Dauvergne il y a presque 5 ans

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.

Révision a9823e5a (diff)
Ajouté par Benjamin Dauvergne il y a presque 5 ans

update french translation (#21908)

Révision d3c163a6 (diff)
Ajouté par Benjamin Dauvergne il y a presque 5 ans

add missing close tag (#21908)

Révision 929b58d9 (diff)
Ajouté par Benjamin Dauvergne il y a presque 5 ans

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

Historique

#1

Mis à jour par Benjamin Dauvergne il y a environ 6 ans

  • Statut changé de Solution déployée à Nouveau

Ne marche pas.

#2

Mis à jour par Benjamin Dauvergne il y a environ 6 ans

  • Version cible changé de 0.10 à 0.11
#3

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

  • Version cible changé de 0.11 à future
#4

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

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

#6

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 ?

#7

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 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

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.

#9

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

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

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.

#11

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

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

#12

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 ?

#13

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

Parce qu'il est méchant.

#15

Mis à jour par Emmanuel Cazenave il y a plus de 5 ans

  • Statut changé de Solution proposée à Solution validée
#16

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
#17

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.

#18

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

#19

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

J'ai oublié un bout.

#20

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

  • Statut changé de Solution proposée à Résolu (à déployer)
#21

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF