Proper use of ldap_connect in conditional statement

Client Joined: Oct 07, 2003 Posts: 735 Location: Ohio

I have a checkbox where I can change the authentication from MySQL to LDAP.

The function ldap_auth sends them to ldap_connect_ex to make the connection to ldap.

If I have multiple ldap servers then what would be the proper way to test for each.

This is the way it is written with just one ldap_host.

Code:

function ldap_connect_ex() {


    global $CBT_config;





    $connection  = ldap_connect($CBT_config['ldap_host'], $CBT_config['ldap_port']);


    ldap_set_option($connection, LDAP_OPT_PROTOCOL_VERSION, 3);





    return $connection;


}

If I also have ldap_host2 then what is the proper way to code this. I believe the return of $connection is boolean.

Would the following work or is there a better way?

Code:

$connection  = ldap_connect($CBT_config['ldap_host'], $CBT_config['ldap_port']);


    if ($connection == false) {


    $connection  = ldap_connect($CBT_config['ldap_host2'], $CBT_config['ldap_port']);


    }

or..

Code:

$connection  = ((ldap_connect($CBT_config['ldap_host'])) || (ldap_connect($CBT_config['ldap_host2'])));

If ldap_host is an actual url such as [ Only registered users can see links on this board! Get registered or login! ] i have read that I don't need to include the ldap_port.

What would be the better way of checking both values?

ldap_host and ldap_host2

Site Admin/Owner Joined: Aug 27, 2002 Posts: 17088

I've never had to program for ldap so I'm shooting in the dark. It seems that either way would work (may need a little tweaking) but you will throw an exception if it can't connect. So, try placing an '@' sign before the ldap_connect (@ldap_connect) to supress the warning/error.