betagouv / rdv-service-public

Prise de RDV pour les services publics
https://rdv.anct.gouv.fr
GNU Affero General Public License v3.0
14 stars 2 forks source link

Erreur callback FranceConnect : "Email est déjà utilisé" #3571

Open francois-ferrandis opened 1 year ago

francois-ferrandis commented 1 year ago

Actuellement, nous avons une dizaine une 60aine de remontées par mois de l'erreur suivante :

La validation a échoué : Email est déjà utilisé (ActiveRecord::RecordInvalid)

Ma supposition : cette erreur remonte lorsqu'un usager a deux comptes en doublon chez nous :

L'usager tente alors de se connecter avec FranceConnect, mais son e-mail renseigné dans FranceConnect est désormais le B. Notre code récupère donc l'usager A (à l'aide de l'ID FranceConnect), et tente de mettre à jour notre usager A avec l'e-mail B, ce qui est impossible car le doublon B est déjà présent dans notre base avec cet e-mail.

Mon avis sur le bug : le volume semble très faible pour le moment et je propose donc de timeboxer l'investigation ou d'ignorer le bug jusqu'à ce que le volume atteigne plus de 5 par semaine.

victormours commented 1 year ago

Les volumes sont assez bas pour qu'on osrte le ticket de l'itération, mais qu'on le laisse en ready

mekaidmekaid commented 11 months ago

le bug persiste : https://github.com/betagouv/rdv-solidarites.fr/issues/3571

Image

Le cas de cet usager ne correspond peut être pas tout à fait à l'hypothèse relevée plus haut. L'usager a déjà un compte mais il est mentionné que celui-ci a été créé la la toute première fois via France Connect en 2022.

sentry-sentry-incubateur-net[bot] commented 2 months ago

Sentry issue: LAPINS-3A

adipasquale commented 2 months ago

le bug persiste et les volumes augmentent

évolution sur les 3 derniers mois : Screenshot 2024-06-24 at 17 28 49

entre 5 et 20 par semaine

on ne peut pas dire combien d’usagers sont impactés c’est moins que ça

sentry-sentry-incubateur-net[bot] commented 1 month ago

Sentry Issue: LAPINS-16K