Development #68656
Erreur lors de l'ajout d'un moyen d'authent OpenId Connect
0%
Description
https://connexion-validation.test.entrouvert.org/manage/authenticators/add/
Je choisis "OpenId Connect", je donne un nom et j'ai une Server Error (500) lorsque je valide.
Fichiers
Révisions associées
Historique
Mis à jour par Pierre Cros il y a plus d'un an
- Sujet changé de Erreur lors de l'ajout d'un moyen d'authent OpenId à Erreur lors de l'ajout d'un moyen d'authent OpenId Connect
Mis à jour par Frédéric Péters il y a plus d'un an
Internal Server Error: /manage/authenticators/add/ IntegrityError at /manage/authenticators/add/ ERREUR: la valeur d'une clé dupliquée rompt la contrainte unique « authentic2_auth_oidc_oidcprovider_issuer_key » DETAIL: La clé « (issuer)=() » existe déjà.
https://sentry.entrouvert.org/entrouvert/publik/issues/90038/
Mis à jour par Frédéric Péters il y a plus d'un an
L'erreur venait ici d'une ancienne (mai 2022) méthode oidc pas configurée (https://connexion-validation.test.entrouvert.org/manage/authenticators/2/detail/, que je viens de supprimer).
Mis à jour par Valentin Deniaud il y a plus d'un an
- Fichier 0001-authenticators-forbid-adding-new-OIDC-provider-if-em.patch 0001-authenticators-forbid-adding-new-OIDC-provider-if-em.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a plus d'un an
C'est tordu, on peut soit ne pas permettre d'issuer vide en le demandant à la création, soit ajouter une condition à l'index d'unicité pour ignorer l'issuer vide, mais pas indiquer que le système est dans un état bancal et que l'utilisateur doit deviner comment réparer sur une autre page.
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Statut changé de Solution proposée à En cours
Mis à jour par Valentin Deniaud il y a plus d'un an
- Fichier 0001-auth_oidc-allow-multiple-oidc-providers-with-empty-i.patch 0001-auth_oidc-allow-multiple-oidc-providers-with-empty-i.patch ajouté
- Statut changé de En cours à Solution proposée
Yep, nettement plus propre en changeant la contrainte d'unicité.
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Mis à jour par Valentin Deniaud il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit ebe0380d256e977704bd1f35ba15d22b5ec491e5 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Wed Oct 19 11:52:24 2022 +0200 auth_oidc: allow multiple oidc providers with empty issuers (#68656)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
auth_oidc: allow multiple oidc providers with empty issuers (#68656)