Development #35283
SAML, utilisateur non sauvé à la fin de AuthenticAdapter.finish_create_user()
Début:
07 août 2019
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Si seul des attributs Django sont touchés, les changements ne seront pas conservés.
Fichiers
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Fichier 0001-auth_saml-implement-attribute-provisionning-after-fi.patch 0001-auth_saml-implement-attribute-provisionning-after-fi.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Bon plutôt que de corriger juste bug j'ai préféré corriger le provisionning
complet parce qu'il n'était fait qu'à la première connexion et ce n'était pas
adéquat. Le test est aussi plus complet car passant par lookup_user()
(peut-être devrais-je aller jusqu'à authenticate).
Mis à jour par Thomas Noël il y a plus de 4 ans
Y'a un "assert False" qui traine à la fin du test, ça aide pas des masses :) Si tu peux repusher la branche, qu'on vérifie que le test est ok (en local ça me semble passer).
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Fichier 0001-auth_saml-implement-attribute-provisionning-after-fi.patch 0001-auth_saml-implement-attribute-provisionning-after-fi.patch ajouté
Oups, voilà.
Mis à jour par Thomas Noël il y a plus de 4 ans
- Statut changé de Solution proposée à Solution validée
Let's go.
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Fichier 0001-auth_saml-implement-attribute-provisionning-after-fi.patch 0001-auth_saml-implement-attribute-provisionning-after-fi.patch ajouté
- Statut changé de Solution validée à Solution proposée
Faute d'orthographe dans la docstring corrigée.
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Statut changé de Solution proposée à Résolu (à déployer)
commit 21257d1a558f82433ba4462b49e97803c100a039 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Wed Aug 7 11:28:35 2019 +0200 auth_saml: implement attribute provisionning after first login (#35283) Also fix bug in finish_create_user() where modified user was not saved.
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
auth_saml: implement attribute provisionning after first login (#35283)
Also fix bug in finish_create_user() where modified user was not saved.