Modify ↓

Opened 6 years ago

Closed 6 years ago

Last modified 3 years ago

#13537 closed enhancement (fixed)

LDAP recursive search

Reported by: dcc@… Owned by: dcc@…
Priority: normal Component: LdapPlugin
Severity: normal Keywords: LDAP
Cc: Trac Release: 0.12

Description (last modified by Ryan J Ollos)

This patch, strongly inspired by the code in #7339, improves LDAP group search in two ways:

  • Instead of getting all the groups, then check for each of them if the user belongs to it, we get the user, and check which groups he is a member of. This is much more efficient for LDAP trees which contain many groups.
  • Once we have the list of groups the user is a direct member of, we check recursively if those groups belong to other groups.

Attachments (1)

trac_ldap_recursive.diff​ (2.0 KB) - added by dcc@… 6 years ago.
patch

Download all attachments as: .zip

Change History (6)

Changed 6 years ago by dcc@…

Attachment: trac_ldap_recursive.diff​ added

patch

comment:1 Changed 6 years ago by Ryan J Ollos

Description: modified (diff)

comment:2 Changed 6 years ago by Ryan J Ollos

Owner: changed from Sylvain Delisle to Ryan J Ollos
Status: new → accepted

comment:3 Changed 6 years ago by Ryan J Ollos

Resolution: → fixed
Status: accepted → closed

In 17390:

LdapPlugin 0.7.0dev: Improve efficiency of LDAP search

Contributed by dcc@…

Fixes #13537.

comment:4 Changed 6 years ago by Ryan J Ollos

Owner: changed from Ryan J Ollos to dcc@…

comment:5 Changed 3 years ago by Ryan J Ollos

In 18457:

LdapPlugin 0.7.0dev: Fix recurrsion resolving groups

Patch by Joachim Mairböck.

Refs #14078, #13537.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain dcc@….
The resolution will be deleted. Next status will be 'reopened'.

Add Comment

E-mail address and name can be saved in the Preferences.

Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.