From b0097ea0a5fbb59570b7553315c3a04935a1e8f8 Mon Sep 17 00:00:00 2001 From: Jan Dittberner Date: Thu, 29 Jan 2015 21:35:33 +0100 Subject: [PATCH] change order of LDAP commands and logging to fix log messages --- gvaldap/ldaptasks/tasks.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/gvaldap/ldaptasks/tasks.py b/gvaldap/ldaptasks/tasks.py index f17a7e6..05c2da8 100644 --- a/gvaldap/ldaptasks/tasks.py +++ b/gvaldap/ldaptasks/tasks.py @@ -97,6 +97,8 @@ def create_ldap_user(username, uid, gid, gecos, homedir, shell, password): if password is not None: ldapuser.set_password(password) _LOGGER.info('set password for LDAP user %s', ldapuser.dn) + ldapuser.save() + _LOGGER.info('LDAP user %s created', ldapuser.dn) if ldapuser.username in ldapgroup.members: _LOGGER.info( 'LDAP user %s is already member of LDAP group %s', @@ -107,8 +109,6 @@ def create_ldap_user(username, uid, gid, gecos, homedir, shell, password): _LOGGER.info( 'LDAP user %s has been added to LDAP group %s', ldapuser.dn, ldapgroup.dn) - ldapuser.save() - _LOGGER.info('LDAP user %s created', ldapuser.dn) return ldapuser.dn