Added support for specifying multiple LDAP servers (branch release/14.0)

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

1 stars 19 forks source link

Added support for specifying multiple LDAP servers (branch release/14.0) #11

Closed chilinux closed 6 years ago

chilinux commented 6 years ago

Updated multiple LDAP server support for branch release/14.0.

This includes updating the function serviceping() accordingly.

It should be noted that serviceping() is a private function so it should only be called inside the scope of the class. However, nothing seems to be calling it. Again, I updated it accordingly in case the call is ever added back but otherwise it might seems logical to remove the function since it seems to currently be dead code?

chilinux commented 6 years ago

I do not seem to have the rights to re-open pull request #10. Do I need to re-submit it as a new PR?

tm1000 commented 6 years ago

@chilinux Thanks Ben. Reopened!