vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

fix: username required

Browse source 
as part of the search the login/user name is required to find the user

issue #13738
This commit is contained in:
Jon Lockwood 2021-07-20 14:09:55 +09:30
parent 622800453a
commit 68a5ac2b34
1 changed files with 1 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
frappe/integrations/doctype/ldap_settings/ldap_settings.py
View file

@ -177,14 +177,12 @@ class LDAPSettings(Document):
fetch_ldap_groups = getattr(user, self.ldap_group_field).values
if ldap_object_class is not None:
conn.search(
search_base=self.organizational_unit_for_groups,
search_filter="(&(objectClass={0})({1}={2}))".format(ldap_object_class,ldap_group_members_attribute, user),
search_filter="(&(objectClass={0})({1}={2}))".format(ldap_object_class,ldap_group_members_attribute, frappe.as_unicode(frappe.form_dict.usr)),
attributes=ldap_attributes) # Build search query
if len(conn.entries) >= 1:
fetch_ldap_groups = []