userPrincipalName should be mail attribute

Our userprincipalnames do not match with mail attribute. TODO: Fix auth with other applications
rinkp · Dec 20, 2020 · 492f15d · 492f15d
1 parent dc4de3a
commit 492f15d
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/syncer.py b/syncer.py
@@ -40,10 +40,10 @@ def sync():
 
     ldap_results = ldap_connector.search_s(config['LDAP_BASE_DN'], ldap.SCOPE_SUBTREE, 
                 config['LDAP_FILTER'], 
-                ['userPrincipalName', 'cn', 'userAccountControl'])
+                ['mail', 'cn', 'userAccountControl'])
 
     ldap_results = map(lambda x: (
-        x[1]['userPrincipalName'][0].decode(),
+        x[1]['mail'][0].decode(),
         x[1]['cn'][0].decode(),
         False if int(x[1]['userAccountControl'][0].decode()) & 0b10 else True), ldap_results)