Produits Entr'ouvert » Authentic 2 » Plugin FS FranceConnect

Ce scénario est bien possible (il est mis en évidence par le test inclus) :
une personne change son mail sur un FI, puis se re-connecte, alors son mail est mis à jour dans Authentic, même s'il introduit un doublon.

Ce patch accepte la connexion, puis si le mail existe déjà ailleurs, il ne prend pas en compte le nouveau mail et afficher un message compréhensible.

(Patch à prendre avec des pincettes car je n'ai pas réussi à déployer la bonne configuration pour le tester localement.)

Même proposition, mais en factorisant un peu de code (plutôt que de le copier/coller).

(et en complétant le test)

Je réalise que l'on passe également dans "apply_user_info_mappings()" quand on s'authentifie via le backend FC, et donc que je dois directement intervenir dans cette fonction.
Rq: bien qu'il y ait 3 points d'entrés différents, je n'en teste que deux (pas encore/ou déjà authentifié), parce que dans le cas "pas encore authentifié" 2 appels se font à la suite :

/src/authentic2_auth_fc/views.py::LoginOrLinkView::get
if request.user.is_authenticated:
   ...
    self.update_user_info(request)  # cas authentifié
    return self.redirect(request)  

user = a2_utils.authenticate( ... # premier appel via le backend (en non authentifié)
 ...
 if user:
 ...
      self.update_user_info(request)
      self.logger.info('logged in using fc sub %s', self.sub)  # deuxième appel en non authentifié
      return self.redirect(request)

Le patch semble répondre au besoin, mais introduit une régression lorsque A2_FC_CREATE=True : le backend va préalablement créer le compte A2 sans mail, si A2_EMAIL_IS_UNIQUE=True et que le mail est déjà utilisé par un autre compte.

Ces 2 options semblent contradictoires : A2_EMAIL_IS_UNIQUE sous entend que l'email est une clé et qu'il existe forcément.
Que faut-il prévoir si ces 2 options sont activées ?

(Je ne vois nul part l'option A2_FC_CREATE, qui est par défaut à False, d'activée en recette ou en production. Où puis-je trouver plus d'information à son sujet ? https://dev.entrouvert.org/issues/10062)

Oui, tu as raison le test fonctionne juste avec cette configuration.
(je sais pas si je ferme ou si je propose juste le test)

(au cas où, patch avec le changement de la configuration par défaut)

Rebasé.
(il s'agit de la version du patch où la correction se fait en configuration)

La nouvelle fixture franceconnect m'a fait réaliser que je ne peux pas me reconnecter avec franceconnect alors que je le suis déjà (ce pourquoi j'ai simplifié le premier test).

J'ai ajouté un test pour exposer mon problème avec A2_EMAIL_IS_UNIQUE.
Comme #50964 prévoit de le supprimer, ce point se solutionnerait tout seul.
(j'ai testé au cas où, #50964 ne corrige pas le bug décrit par ce ticket)

À présent quand si un compte FC usurpe le mail d'un autre compte FC, alors on ne met plus à jour le compte Publik concerné mais le compte Publik qui possède le mail.

Je pense que ce ticket est obsolète et serais d'avis de le fermer : le backend FC ayant évolué, les patch proposés ici n'apportent plus rien.

Je fournis cependant 2 tests qui permettront éventuellement de repartir sur un nouvel intitulé de ticket.

Merci Benjamin pour la confirmation du bug.

0001 pour réparer le bug : si on arrive avec le sub 1234 et qu'il existe une liaison pour ce sub on ne doit pas toucher à l'utilisateur avec le sub 4567.
0002 qui reprend où on en était jusqu'à présent.

Note : j'ai délibérément zappé https://dev.entrouvert.org/issues/45199#note-13 pour me concentrer en premier sur le test qui expose le problème.
(il y a des configuration modifiées en productions qui ne verront pas la nouvelle configuration par défaut)

Enfin,

c'est l'occasion de changer le mapping par défaut, une fois qu'on a un mail on ne le change pas, même s'il vient de FC

Je préférerais que ce soit l'objet d'un nouveau ticket (le sujet me dépasse), sinon je n'arriverai jamais à fermer proprement celui-là.
Après j’entends bien que ce nouveau paradigme est censé remplacer 0002.
(Pour moi il faudrait corriger 0001 dans un ticket dédié et en ouvrir un autre qui permette de partir sur cette nouvelle voie.)

0001: entre temps le bug mentionné ci-dessus a été corrigé, comme le montre le test.
0002: si le mail est déjà présent on y touche jamais (mais si le mail vient de FC on le coche vérifié).

0002 retire le second test qui devient redondant avec le premier.

Ok.

commit 019159d30e510f1a2579253beea75cedb7ac1ab3
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Fri May 14 18:09:19 2021 +0200

    auth_fc: do not update user email with email returned by FC (#45199)

commit f148399793ccb163a536eb2ae7ae2e2315e57d99
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Fri May 14 17:34:48 2021 +0200

    test: auth_fc updates the user email with FC info (#45199)