search

percussion / percussioncms

Percussion CMS - Content Management System
https://percussioncmshelp.intsof.com/percussioncms
Apache License 2.0
12 stars 9 forks source link

Add support for multiple Active Directory / LDAP connections in rxconfig/LDAP/ldapserver.xml #562

Closed natechadwick closed 2 years ago

natechadwick commented 2 years ago

Currently you can not have multiple xml blocks in the ldapserver.xml (in order to pull users from two different Active Directory servers). The file is not read and the config.xml is not updated if there is more than one LDAP connection in the ldapserver.xml.

Proposed Solution:- Remove ldapServer.xml and use the existing config.xml instead that is managed by the Server Administrator tool instead.

natechadwick commented 2 years ago

There is a work around for this where you can edit the config.xml with server admin tool directly to support multiple directories it has always supported this. Removing the ldapserver.xml configuration and process makes sense to simplify things. The code that would need changed is mainly here:

https://github.com/percussion/percussioncms/blob/4f3b8d1aae76abae7e8026b8e3a3acb9b96f2e06/projects/sitemanage/src/main/java/com/percussion/user/service/impl/PSDirectoryServiceConfig.java

natechadwick commented 2 years ago

We should also remove the directory on upgrade.