#33850 [Asn]: [PATCH]: Support LDAP connection timeouts

From: Date: Tue, 04 Oct 2005 03:10:29 +0000
Subject: #33850 [Asn]: [PATCH]: Support LDAP connection timeouts
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-85962@lists.php.net to get a copy of this message
ID: 33850 User updated by: simon dot kissane at mq dot edu dot au -Summary: [PATCH]: Support LDAP_X_OPT_CONNECT_TIMEOUT -Reported By: skissane at gmail dot com +Reported By: simon dot kissane at mq dot edu dot au Status: Assigned Bug Type: Feature/Change Request Operating System: * PHP Version: 5CVS, 4CVS (2005-07-27) Assigned To: sniper New Comment: I have written a new version of this patch. This supports, as well of Netscape's LDAP_X_OPT_CONNECT_TIMEOUT, OpenLDAP's LDAP_OPT_TIMEOUT & LDAP_OPT_NETWORK_TIMEOUT. Note that these take a struct timeval, whereas Netscape takes an integer. I have chosen to represent struct timeval as an object with two properties (tv_sec & tv_usec, corresponding to the struct timeval fields of the same name.) New version of patch (against 5.0.5) is available here: http://www.mq.edu.au/~skissane/ldap-timeout-5.0.5.patch Previous Comments: ------------------------------------------------------------------------ [2005-07-25 08:55:10] simon dot kissane at mq dot edu dot au Description: ------------ I have written a patch to support LDAP_X_OPT_CONNECT_TIMEOUT (which is defined by the Netscape LDAP C SDK). This required also changing ldap_connect to call ldap_init instead of ldap_open (but only if LDAP_X_OPT_CONNECT_TIMEOUT is defined), which is necessary if LDAP_X_OPT_CONNECT_TIMEOUT is to do anything. In any case, ldap_open is deprecated, so PHP shouldn't be calling it unless necessary. Reproduce code: --------------- http://www.mq.edu.au/~skissane/ldap-nsldap-timeout.patch Expected result: ---------------- N/A Actual result: -------------- N/A ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/?id=33850&edit=1

« previous php.bugs (#85962) next »