From 0a1b79d538c78b8afdf0e19e4a8ae53b1ca831d5 Mon Sep 17 00:00:00 2001 From: Liam Middlebrook Date: Wed, 28 Sep 2016 17:08:50 -0400 Subject: [PATCH] Fix LDAP set field NoneType Fixes #59 --- conditional/util/ldap.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/conditional/util/ldap.py b/conditional/util/ldap.py index a3ad653c..2e30ff23 100644 --- a/conditional/util/ldap.py +++ b/conditional/util/ldap.py @@ -64,7 +64,10 @@ def __ldap_set_field__(username, field, new_val): % username) old_result = ldap_results[0][1] new_result = copy.deepcopy(ldap_results[0][1]) - new_result[field] = [str(new_val).encode('ascii')] + if new_val is not None: + new_result[field] = [str(new_val).encode('ascii')] + else: + new_result[field] = [None] ldap_mod_list = ldap.modlist.modifyModlist(old_result, new_result) userdn = "uid=%s,%s" % (username, user_search_ou) ldap_conn.modify_s(userdn, ldap_mod_list)