Req #69471 [Asn->Csd]: php-ldap is using deprecated ldap functions
Edit report at https://bugs.php.net/bug.php?id=69471&edit=1
ID: 69471
Updated by: heiglandreas@php.net
Reported by: come dot bernigaud at opensides dot be
Summary: php-ldap is using deprecated ldap functions
-Status: Assigned
+Status: Closed
Type: Feature/Change Request
Package: LDAP related
PHP Version: 5.6.8RC1
Assigned To: mcmic
Block user comment: N
Private report: N
New Comment:
From what i've seen this issue is solved in the current releases of PHP.
So I'm closing it.
Previous Comments:
------------------------------------------------------------------------
[2015-11-03 11:19:36] blizzz at owncloud dot com
Thank you for responding. Meanwhile I also investigated more in our (ownCloud) case and found it was
a bug with us that on one place we just added the port to the host without verifying whether a URL
was given. I apologize for not posting here after fixing this. From what I can tell, it indeed looks
good and the patch is fine, our usage was not. Sorry for troubling you.
------------------------------------------------------------------------
[2015-11-03 04:56:04] mcmic@php.net
Ok, some of the bug report you mentioned were misleading, what is broken is using host:port, using a
full ldap URI like ldap://host:port works as expected.
Iâm gonna ask on the ML but this does not seem like a big regression to me as the use of
host:port was not tested by the tests and not authorized by the documentation. (But it does seem to
break existing code, so Iâm not sure)
------------------------------------------------------------------------
[2015-11-03 04:10:54] mcmic@php.net
It definitely seems related, this was not intended.
What I donât get is that there is a test for this: https://github.com/php/php-src/blob/PHP-5.6/ext/ldap/tests/ldap_connect_variation.phpt
And at the time it was passing. Iâm gonna build last git and try to see if it passes.
------------------------------------------------------------------------
[2015-10-29 11:31:11] blizzz at owncloud dot com
Is this related to https://github.com/php/php-src/commit/e7af0fe1eb89e40671e86a588aa1b78607b85461
?
It turns out that some usages of ldap_connect now fail to work (e.g. https://www.netways.org/issues/2931,
https://dev.icinga.org/issues/9298, https://github.com/owncloud/core/issues/20020).
Is this as intended?
------------------------------------------------------------------------
[2015-06-17 13:54:23] mcmic@php.net
This first patch has been pushed.
All obsolete functions should be removed so that we can remove the -DLDAP_DEPRECATED=1 option
------------------------------------------------------------------------
The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
https://bugs.php.net/bug.php?id=69471
--
Edit this bug report at https://bugs.php.net/bug.php?id=69471&edit=1
Thread (8 messages)