Produits Entr'ouvert » Authentic 2

• créant l'utilisateur immédiatement
• connexion de cet utilisateur
• vérification de la présence d'attributs requis vides
  • 1. si oui, redirection vers la page d'édition du profil en conservant le next_url initial
  • 2. si non, redirection vers le next_url initial

On peut ajouter un setting pour dire qu'on veut le comportement 1. dans tous les cas.

... et pour régler le cas "cocher la case CGU", avoir un mapping configurable des attributs, qui poserait un "True" sur la case à cocher quand on arrive de FC ? (Parce que sinon cet attribut ne sera jamais là et on arrivera toujours sur le comportement 1)

Je vous suis.

Je me rends compte que lors de la création du compte via FC, le token contient (testé sur la recette d'Arles):

{'next': '/fc/callback/?next=/idp/saml2/continue%3Fnonce%3D_C8CA30D5B74232D5440E4C33EEA54BC2',
 'email': 'test@yopmail.com',
 'title': 'female',
 'last_name': 'DURANTÉÀÈÙҌƌÆ',
 'first_name': 'Julieéàèùòœæœæ',
 'no_password': True,
 'valid_email': False,
 'confirm_data': 'required',
 'franceconnect': True,
 'lieu_de_naissance': '2B002',
 'pays_de_naissance': '99100',
 'authentication_method': 'france-connect',
 'nom_dusage_si_different': ''}

Cela fait qu'après la completion des infos usager, il est de nouveau rédirigé vers FC, qui demande la confirmation pour continuer vers Publik, au lieu d'être loggué et avoir les messages "Votre compte a été créé" et "Votre compte a été lié au compte FC ...".

Il doit y avoir quelque chose que a changé côté FranceConnect car je n'avais pas ce comportement la semaine dernière.

On s'est mal compris quelque part, le but n'est pas de complexifier la vue d'enregistrement mais de la débrancher du module FranceConnect totalement, et de créer directement l'utilisateur. Il ne devrait y avoir aucune modification en dehors de authentic2_auth_fc.

Benjamin Dauvergne a écrit :

Il faut en fait ajouter la création du compte sans condition dans le backend

Cela veut dire que le setting A2_FC_CREATE perd son sens et peut être viré. On est d'accord?

Serghei Mihai a écrit :

Cela veut dire que le setting A2_FC_CREATE perd son sens et peut être viré. On est d'accord?

Oui.

J'ai pas relu encore mais j'adore tout ce rouge dans le patch :)

            user_filter = {'deleted__isnull': True}

deleted n'existe plus. En fait tout le code autour de user_filter m'a l'air de traîner d'avant au lieu d'être complètement déplacé dans un block "on a un email et il doit être unique".

Benjamin Dauvergne a écrit :

[...]

deleted n'existe plus.

D'acc.

En fait tout le code autour de user_filter m'a l'air de traîner d'avant au lieu d'être complètement déplacé dans un block "on a un email et il doit être unique".

La même logique de recherche s'applique à 2 endroits différents, dans le backend et la vue. Je vais voir comment factoriser ça.