Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Estended LDAP integration to allow multiple users search filter. #61

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

mmariuzzo
Copy link

This is useful, for example, when your users are inside different
organizationUnit and your LDAP server is ActiveDirectory.
AD doesn't support complex search query

I've introduced new sections inside config.ini like this one

[Users:ldap:1]
; A free text name for this extra query
Alias=italy
; The organization unit where the users are
BaseDN=OU=italy,DC=insanefactory,DC=com
; The filter, which identifies the object as a user object.
SearchFilter=(&(objectClass=person)(objectClass=user))

This is useful, for example, when your users are inside different
organizationUnit and your LDAP server is ActiveDirectory.
AD doesn't support complex search query
…, maybe, some contents.

I've extended the "repository structure" concept allowing to use a dump as initial structure

You have to add custom sections in your config.ini like this one

[Repositories:template:1]
; free text structure name
Name=Liferay SDK Structure
; absolute or relative path to file to load
Source=./data/templates/test01.dump

WARNING: You have to use dump created with the same version of SVN (or campatible with it)
@mfreiholz mfreiholz added this to the Version 1.7.0 milestone Mar 26, 2014
@mfreiholz mfreiholz self-assigned this Mar 26, 2014
Inside config.ini you can use attributes properties to retrieve "sn" and/or
"givenName" from LDAP and expose it on UserList
@mundtl
Copy link

mundtl commented Dec 14, 2023

We recently had to install this to work around LDAP performance issues.
Works like a charm! <3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants