Bug #57039
ldap: sync-ldap-users
Début:
17 septembre 2021
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Suite à #54078 on a des erreurs dans les cas ou un user n'a pas pu être créé car attributs LDAP nécessaires à la création du compte manquants (cf #57037).
Fichiers
Révisions associées
Historique
Mis à jour par Serghei Mihai il y a plus de 2 ans
Dans:
user = backend._return_user(dn, None, conn, block, attrs)
if user._changed or user._created:
....
on ne vérifie pas si le user est créé.
Je pense qu'il manque:
user = backend._return_user(dn, None, conn, block, attrs)
if not user:
continue
if user._changed or user._created:
....
Mis à jour par Paul Marillonnet il y a plus de 2 ans
- Statut changé de Nouveau à En cours
- Assigné à mis à Paul Marillonnet
Mis à jour par Serghei Mihai il y a plus de 2 ans
- Statut changé de En cours à Nouveau
- Assigné à
Paul Marillonnetsupprimé
Etant donné qu'on a déjà la ligne:
retrieved attributes for 'cn=invité,cn=users,dc=atdpgd,dc=local': {'dn': 'CN=Invité,CN=Users,DC=atdpgd,DC=local', 'cn': ['Invité'], 'samaccountname': ['Invité']}
je pense qu'on peut limiter le message de warning à:
log.warning('unable to retrieve user for dn %s' % dn)
Mis à jour par Paul Marillonnet il y a plus de 2 ans
- Fichier 0001-ldap-check-whether-backend-performed-actual-user-ret.patch 0001-ldap-check-whether-backend-performed-actual-user-ret.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Serghei Mihai a écrit :
Etant donné qu'on a déjà la ligne:
[...]je pense qu'on peut limiter le message de warning à:
[...]
Ok.
Mis à jour par Serghei Mihai il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
- Assigné à mis à Paul Marillonnet
Mis à jour par Paul Marillonnet il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 82a1a16430e904d65775836451f6b2bb4e6168b8
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date: Fri Sep 17 11:47:26 2021 +0200
ldap: check whether backend performed actual user retrieval (#57039)
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
Ça aurait du être "unable to create user" pas "retrive", on a bien récupéré l'enregistrement correspondant, d'ailleurs on le dit plus haut visiblement (en debug j'espère).
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
ldap: check whether backend performed actual user retrieval (#57039)