php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Request #69445 Support for LDAP EXOP operations
Submitted: 2015-04-14 13:11 UTC Modified: 2017-07-05 14:38 UTC
Votes:16
Avg. Score:4.7 ± 0.7
Reproduced:15 of 15 (100.0%)
Same Version:13 (86.7%)
Same OS:10 (66.7%)
From: come dot bernigaud at opensides dot be Assigned: mcmic (profile)
Status: Closed Package: LDAP related
PHP Version: Irrelevant OS:
Private report: No CVE-ID: None
 [2015-04-14 13:11 UTC] come dot bernigaud at opensides dot be
Description:
------------
PHP-LDAP should allow to use extended operations.

This would allow a clean password change even when ppolicy overlay is used for instance.
Some patches for including this in PHP exists: http://www.openldap.org/lists/openldap-technical/201504/msg00062.html
After some research it seems Pierangelo Masarati had made a patch for adding EXOP to PHP-LDAP ten years ago.
In the link there is an updated version of this patch.

Any chance of something like this getting merged upstream?


Patches

ldap-ctrl-exop56.patch-manu_at_netbsd_dot_org (last revision 2015-09-28 15:39 UTC) by manu at netbsd dot org)

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2015-06-10 12:54 UTC] mcmic@php.net
-Assigned To: +Assigned To: mcmic
 [2017-01-10 07:50 UTC] heiglandreas@php.net
Are you still working on this?
 [2017-01-10 14:34 UTC] heiglandreas@php.net
-Assigned To: mcmic +Assigned To:
 [2017-03-06 17:51 UTC] 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.
 [2017-03-09 09:20 UTC] 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-06-17 08:56 UTC] 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-06-17 11:14 UTC] 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-19 08:35 UTC] 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-07-05 14:38 UTC] mcmic@php.net
-Status: Open +Status: Closed -Assigned To: +Assigned To: mcmic
 [2017-07-05 14:38 UTC] mcmic@php.net
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.


 
PHP Copyright © 2001-2017 The PHP Group
All rights reserved.
Last updated: Sun Nov 19 01:31:42 2017 UTC