Hello
Quite some time ago, Pierangelo Masarati wrote a patch to brind ctrl and exop support to PHP. Unfortunately the patch was never checked in upstream and broke with PHP 5.3
I trieed to upgrade the patch for PHP 5.3 support. Here is my first attempt, which builds on PHP 5.3.17: http://ftp.espci.fr/shadow/manu/ldap-ctrl-exop-20121025.patch
I successfully tested ldap_exop_passwd, ldap_exop_whoami, and ldap_refresh.
Everything else was not tested and is broken without any doubt (zend_parse_parameters 3rd argument is wrong, it should probably be Z more often).
Feel free to help, either by fixing the remaining bits, or by providing a test case for the broken parts (I never used controls and therefore I am not going to do it).
Emmanuel Dreyfus manu@netbsd.org wrote:
http://ftp.espci.fr/shadow/manu/ldap-ctrl-exop-20121025.patch
I successfully tested ldap_exop_passwd, ldap_exop_whoami, and ldap_refresh.
Everything else was not tested and is broken without any doubt (zend_parse_parameters 3rd argument is wrong, it should probably be Z more often).
Here is an updated version with zend_parse_parameters calls for ctrl probably fixed. One still have to test it out, I did not: http://ftp.espci.fr/shadow/manu/ldap-ctrl-exop-20121026.patch
26.10.2012 07:49, Emmanuel Dreyfus пишет:
Emmanuel Dreyfus manu@netbsd.org wrote:
http://ftp.espci.fr/shadow/manu/ldap-ctrl-exop-20121025.patch
I successfully tested ldap_exop_passwd, ldap_exop_whoami, and ldap_refresh.
Everything else was not tested and is broken without any doubt (zend_parse_parameters 3rd argument is wrong, it should probably be Z more often).
Here is an updated version with zend_parse_parameters calls for ctrl probably fixed. One still have to test it out, I did not: http://ftp.espci.fr/shadow/manu/ldap-ctrl-exop-20121026.patch
Have you sent it to PHP lists? I doubt the code is for openldap tree.
--On Sunday, October 28, 2012 2:32 PM +0400 Roman Rybalko openldap@romanr.name wrote:
Have you sent it to PHP lists? I doubt the code is for openldap tree.
The PHP development team has ignored fixes to their horribly broken LDAP API implementation for years. I don't see why anyone would expect them to reverse course now.
--Quanah
--
Quanah Gibson-Mount Sr. Member of Technical Staff Zimbra, Inc A Division of VMware, Inc. -------------------- Zimbra :: the leader in open source messaging and collaboration
openldap-technical@openldap.org
-
Emmanuel Dreyfus -
manu＠netbsd.org
-
Quanah Gibson-Mount -
Roman Rybalko