Bug #5835
le backend ldap ne retourne la bonne association
100%
Description
Le backend ldap devrait toujours retourner l'association qui est la dernière à avoir été utilisée.
Fichiers
Révisions associées
ldap backend: use a more pythonic sort to get the last connected
Closes #5835
Historique
Mis à jour par Jérôme Schneider il y a plus de 9 ans
- Fichier 0001-ldap-backend-always-get-the-last-connected-associati.patch 0001-ldap-backend-always-get-the-last-connected-associati.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus de 9 ans
Pourquoi ne pas simplement trier selon lastConnectionDate ?
def cmp_reverse_last_connection_date(x, y): return -cmp(x[1]['lastConnectionDate'][0], y[1]['lastConnectionDate'][0])
Pour ensuite n'avoir qu'à ajouter avant les deux endroits où results[0]
était utilisé, results.sort(cmp_reverse_last_connection_date)
.
Cela étant, curieux, ça arrive dans quelles situations d'avoir plusieurs associations pour le même tuple (sp_name, sp_login, idp_unique_id, idp_name) ?
Mis à jour par Jérôme Schneider il y a plus de 9 ans
Cela étant, curieux, ça arrive dans quelles situations d'avoir plusieurs associations pour le même tuple (sp_name, sp_login, idp_unique_id, idp_name) ?
C'est utilisé pour la gestion du multi compte sur la médiathèque de Vincennes. Un bug permettait également d'avoir une multi association pour un site à mono association. Il fallait pour cela avoir un compte valide qui devienne invalide et saisir un nouveau compte valide mais avec un login différent. On se retrouvait alors avec deux associations.
Mis à jour par Jérôme Schneider il y a plus de 9 ans
- Fichier 0001-ldap-backend-use-a-more-pythonic-sort-to-get-the-las.patch 0001-ldap-backend-use-a-more-pythonic-sort-to-get-the-las.patch ajouté
J'avais commité par erreur mon ancien patch. Voici un nouveau qui tient compte des remarques.
Mis à jour par Jérôme Schneider il y a plus de 9 ans
- Statut changé de En cours à Résolu (à déployer)
- % réalisé changé de 0 à 100
Appliqué par commit commit:d21f26c66ba7232e0c684882f65ae33d163db048.
Mis à jour par Jérôme Schneider il y a plus de 9 ans
Appliqué par commit commit:2f6d1c94a15c7d8f6761dbf2f2d7c39db9c1e2de.
ldap backend: use a more pythonic sort to get the last connected
Closes #5835