Development #48174
auth_oidc: revoir les index sur les comptes OIDC
0%
Description
Actuellement il y a index d'unicité sur la colonne user et la colonne sub, ça rend tout à fait impossible l'éventuelle liaison d'un utilisateur à plusieurs fournisseurs mais surtout la réutilisation d'un même sub sur plusieurs fournisseurs (plus probable si c'est un simple entier par exemple).
Fichiers
Demandes liées
Révisions associées
auth_oidc: normalize unicode strings (#48174)
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Fichier 0001-auth_oidc-make-account-unique-on-provider-user-and-p.patch 0001-auth_oidc-make-account-unique-on-provider-user-and-p.patch ajouté
- Fichier 0002-auth_oidc-normalize-unicode-strings-48174.patch 0002-auth_oidc-normalize-unicode-strings-48174.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Sujet changé de auth_oidc: revoir les indexes sur les comptes OIDC à auth_oidc: revoir les index sur les comptes OIDC
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Bloque Development #48168: conserver trace des liaisons externes quand un utilisateur est supprimé ajouté
Mis à jour par Emmanuel Cazenave il y a plus de 3 ans
"l'éventuelle liaison d'un utilisateur à plusieurs fournisseurs", c'est réaliste ?
De loin j'ai l'impression qu'il y aurait sûrement beaucoup d'autres choses à revoir que la seule contrainte en DB pour que ça puisse marcher, voir même que ce n'est pas souhaitable de commencer à ouvrir cette porte, un compte local par IDP, c'est simple et compréhensible.
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Fichier 0001-auth_oidc-make-account-unique-on-provider-user-and-p.patch 0001-auth_oidc-make-account-unique-on-provider-user-and-p.patch ajouté
- Fichier 0002-auth_oidc-normalize-unicode-strings-48174.patch 0002-auth_oidc-normalize-unicode-strings-48174.patch ajouté
Ok, mais le deuxième est possible.
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Fichier 0001-auth_oidc-make-account-unique-on-provider-user-and-p.patch 0001-auth_oidc-make-account-unique-on-provider-user-and-p.patch ajouté
- Fichier 0002-auth_oidc-normalize-unicode-strings-48174.patch 0002-auth_oidc-normalize-unicode-strings-48174.patch ajouté
Avec le bon test.
Mis à jour par Emmanuel Cazenave il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 8b89b7cadc2b9d4be99e61ffa1ea847e917c9128 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Mon Nov 2 13:58:41 2020 +0100 auth_oidc: normalize unicode strings (#48174) commit 5d28c9034cce6d61e50b0ad7ed7a45cbf69cb002 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Mon Nov 2 11:43:33 2020 +0100 auth_oidc: make account unique on (provider, user) and (provider, sub) (#48174)
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
auth_oidc: make account unique on (provider, user) and (provider, sub) (#48174)