Req #69445 [Opn->Csd]: Support for LDAP EXOP operations
Edit report at https://bugs.php.net/bug.php?id=69445&edit=1
ID: 69445
Updated by: mcmic@php.net
Reported by: come dot bernigaud at opensides dot be
Summary: Support for LDAP EXOP operations
-Status: Open
+Status: Closed
Type: Feature/Change Request
Package: LDAP related
PHP Version: Irrelevant
-Assigned To:
+Assigned To: mcmic
Block user comment: N
Private report: N
New Comment:
The fix for this bug has been committed.
Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.
For Windows:
http://windows.php.net/snapshots/
Thank you for the report, and for helping us make PHP better.
Previous Comments:
------------------------------------------------------------------------
[2017-06-19 08:35:33] mcmic@php.net
Glad to see there is an up to date patch for this.
As said by Andreas it would need to be split at least in two, controls and EXOP.
Also we need to look closely into the resulting API to see if it makes sense with what is already in
PHP-LDAP. (And if there is any way to avoid duplicating all methods with an _ext version I would
prefer that)
------------------------------------------------------------------------
[2017-06-17 11:14:48] heiglandreas@php.net
From what I see from a first glance this patch introduces handling of LDAP-Controls. I'd rather
see that splitted into at least two patches. One for handling LDAP-Controls and the second one to
use those controls to handle the ppolicy-things.
I will see to have a more detailed look during the next weekâ¦
------------------------------------------------------------------------
[2017-06-17 08:56:10] post at rolandgruber dot de
What about this?
http://cvsweb.netbsd.org/bsdweb.cgi/pkgsrc/databases/php-ldap/files/ldap-ctrl-exop56.patch
This patch is two weeks old. Can you take a look?
------------------------------------------------------------------------
[2017-03-09 09:20:56] mcmic@php.net
As you can expect the patch is outdated.
Itâs also huge and seem to contain stuff not related to EXOP.
It would need to be updated, split in small pieces, and tested.
------------------------------------------------------------------------
[2017-03-06 17:51:06] post at rolandgruber dot de
Is there any issue with the patch why it is not applied? It is a shame if patches exist for 10 years
and do not get included.
------------------------------------------------------------------------
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=69445
--
Edit this bug report at https://bugs.php.net/bug.php?id=69445&edit=1
Thread (7 messages)