search

FreePBX / userman

Module of FreePBX (User Management) :: This Module manages all PBX users. It allows an Administrator to create a User Account that can be used by other applications for login or API purposes.
http://www.freepbx.org
GNU Affero General Public License v3.0
2 stars 20 forks source link

Added multi-ldap host support #10

Closed chilinux closed 6 years ago

chilinux commented 6 years ago

This is an attempt to leverage OpenLDAP v2's ability to specify multiple LDAP URIs to improve the resilience in a way that specifying just a single server can not. The serviceping function will return successful as long as at least one specified host access a socket connection. The buildldapuri will then use a space or comma delimited list of hosts along with the specified connection protocol and port to build a list or LDAP URIs. This string should then work for being passed to PHP's ldap_connect and ldapsearch with the -H flag.

Without these changes, only a single host can be specified and any problem connecting or binding to that single host will result in the LDAP call failing despite that the environment may provide replication to additional LDAP servers to provide high availability of the LDAP service.

tm1000 commented 6 years ago

Approved CLA

tm1000 commented 6 years ago

I can't accept this unless the same work is done in 14. As if I don't a feature will be in 13 but not in 14.

You can reopen this PR when and if you do this. Thank you for your work

chilinux commented 6 years ago

Submitted pull request #11 to address the branch release/14.0

tm1000 commented 6 years ago

See: https://issues.freepbx.org/browse/FREEPBX-17848