Development #65412
auth_oidc: locker le sub pendant la création de compte/liaison
0%
Fichiers
Révisions associées
auth_oidc: get a lock on the sub during authentication (#65412)
Historique
Mis à jour par Benjamin Dauvergne il y a presque 2 ans
- Fichier 0002-auth_oidc-get-a-lock-on-the-sub-during-authenticatio.patch 0002-auth_oidc-get-a-lock-on-the-sub-during-authenticatio.patch ajouté
- Fichier 0001-misc-add-Lock.lock_identifier-method-65412.patch 0001-misc-add-Lock.lock_identifier-method-65412.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Paul Marillonnet il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
J’ai souvenir d’un problème concret qui justifiait ce ticket, mais les congés sont passés entre temps et je ne retrouve plus, ma faute. Les patches me conviennent, ack.
Mis à jour par Benjamin Dauvergne il y a plus d'un an
Ça évite des erreurs de non unicité à l'insertion du FcAccount qui génère une 500 pour l'utilisateur (sans que je sache pourquoi des fois les gens arrivent à produire plusieurs requêtes de callback OIDC, la première réussie mais les gens sont en fait dans la deuxième qui échoue, avec un lock on s'assure qu'une seule ne réussisse avant de tenter de chercher une liaison et donc l'insertion n'échoue jamais parce qu'on ne la fait plus). J'ai surtout vu ça sur FranceConnect je crois mais tant qu'à faire.
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit 55a2c56130f65d25036410ff402824c9e17898b8
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Thu Jul 7 13:05:56 2022 +0200
auth_oidc: get a lock on the sub during authentication (#65412)
commit e6d3cbe80894b27793c86240cdf0dcac35d72adb
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Thu Jul 7 12:57:25 2022 +0200
misc: add Lock.lock_identifier method (#65412)Mis à jour par Paul Marillonnet il y a plus d'un an
Benjamin Dauvergne a écrit :
Ça évite des erreurs de non unicité à l'insertion du FcAccount qui génère une 500 pour l'utilisateur (sans que je sache pourquoi des fois les gens arrivent à produire plusieurs requêtes de callback OIDC, la première réussie mais les gens sont en fait dans la deuxième qui échoue, avec un lock on s'assure qu'une seule ne réussisse avant de tenter de chercher une liaison et donc l'insertion n'échoue jamais parce qu'on ne la fait plus). J'ai surtout vu ça sur FranceConnect je crois mais tant qu'à faire.
D’ac oui c’est vrai, ça m’est revenu en étudiant l’affaire pour #65411. Merci pour l’historique, que je n’avais plus en tête au moment de relire ce patche.
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
misc: add Lock.lock_identifier method (#65412)