Deprecate building ext/ldap against Oracle LDAP

php.internals

Christoph Becker

1 year ago
Hi all, there has been some discussion about this in a GH issue[1]. TL;DR: building ext/ldap against Oracle LDAP had been supported, and is theoretically still supported, but is apparently broken for a while. The Oracle LDAP implementation is part of Oracle Instant Client, and uses an older LDAP API; apparently, there are no plans for updating this. So users are almost certainly better off to build against OpenLDAP (and if they need OCI8, to build that as shared library, and load after ext/ldap). However, so far no further steps have been taken. Since it appears that there will be a PHP 8.5, I think it makes sense to deprecate building against Oracle LDAP (instead of silently dropping in PHP 9). I should note that the ldap_connect_wallet() function, available only as of PHP 8.3[2] would also be part of the deprecation, as well as other existing Oracle LDAP specific features. Thoughts? [1] <https://github.com/php/php-src/issues/15051> [2] <https://wiki.php.net/rfc/deprecate_functions_with_overloaded_signatures#ldap_connect> Christoph

Christoph Becker

1 year ago
On 09.11.2024 at 17:18, Christoph M. Becker wrote:
> there has been some discussion about this in a GH issue[1]. > > TL;DR: building ext/ldap against Oracle LDAP had been supported, and is > theoretically still supported, but is apparently broken for a while. > The Oracle LDAP implementation is part of Oracle Instant Client, and > uses an older LDAP API; apparently, there are no plans for updating > this. So users are almost certainly better off to build against > OpenLDAP (and if they need OCI8, to build that as shared library, and > load after ext/ldap). > > However, so far no further steps have been taken. Since it appears that > there will be a PHP 8.5, I think it makes sense to deprecate building > against Oracle LDAP (instead of silently dropping in PHP 9). > > I should note that the ldap_connect_wallet() function, available only as > of PHP 8.3[2] would also be part of the deprecation, as well as other > existing Oracle LDAP specific features. > > Thoughts? > > [1] <https://github.com/php/php-src/issues/15051> > [2] > <https://wiki.php.net/rfc/deprecate_functions_with_overloaded_signatures#ldap_connect>
Since there has been no feedback so far, I would like to go on with a deprecation RFC. As this deprecation doesn't seem to be highly controversial, nor requires an eleborate justification, I like to request permission to add it to the general "Deprecations for PHP 8.5" RFC[1] which is still in draft status (although it erroneously says "Pending Implementation"). [1] <https://wiki.php.net/rfc/deprecations_php_8_5> Christoph

Tim Düsterhus

1 year ago
Hi Am 2025-02-06 23:24, schrieb Christoph M. Becker:
> request permission to add it to the general "Deprecations for PHP 8.5" > RFC[1] which is still in draft status (although it erroneously says > "Pending Implementation").
Just go ahead and make sure to add your name to the section you add. We'll figure it out from there when the RFC actually goes to discussion. That's how it went for 8.3 and 8.4 as well. The “Status” I've just fixed. Best regards Tim Düsterhus

Christoph Becker

1 year ago
On 07.02.2025 at 14:47, Tim Düsterhus wrote:
> Am 2025-02-06 23:24, schrieb Christoph M. Becker: >> request permission to add it to the general "Deprecations for PHP 8.5" >> RFC[1] which is still in draft status (although it erroneously says >> "Pending Implementation"). > > Just go ahead and make sure to add your name to the section you add. > We'll figure it out from there when the RFC actually goes to discussion. > That's how it went for 8.3 and 8.4 as well.
Thank you! Done: <https://wiki.php.net/rfc/deprecations_php_8_5?do=diff&rev2%5B0%5D=1739053373&rev2%5B1%5D=1740137451&difftype=sidebyside>. Christoph