Doc #76546 [Com]: ldap_exop_refresh function not available in 7.2
| From: | post at rolandgruber dot de | Date: | Tue, 03 Jul 2018 16:41:59 +0000 |
| Subject: | Doc #76546 [Com]: ldap_exop_refresh function not available in 7.2 | ||
| References: | 1 | Groups: | php.doc.bugs |
| Request: | Send a blank email to doc-bugs+get-15845@lists.php.net to get a copy of this message | ||
Edit report at https://bugs.php.net/bug.php?id=76546&edit=1
ID: 76546
Comment by: post at rolandgruber dot de
Reported by: post at rolandgruber dot de
Summary: ldap_exop_refresh function not available in 7.2
Status: Verified
Type: Documentation Problem
Package: LDAP related
Operating System: Linux
PHP Version: 7.2.7
Assigned To: mcmic
Block user comment: N
Private report: N
New Comment:
True, I was in wrong branch. The code is not part of 7.2. So documentation error only.
For anybody who faces the same issue here is a workaround for 7.2:
$dn= 'cn=user,dc=test,dc=org';
$ttl = 3600;
$data = '';
$data .= '80' . sprintf("%'.02x", strlen($dn)) . bin2hex($dn);
$data .= '81' . sprintf("%'.02x", 4) . sprintf("%'.08x",
$ttl);
$data = hex2bin('30' . sprintf("%'.02x", strlen($data)/2) . $data);
$success = @ldap_exop($ldapHandle, LDAP_EXOP_REFRESH, $data);
Previous Comments:
------------------------------------------------------------------------
[2018-07-03 14:48:35] cmb@php.net
> [â¦] or maybe I missed the deadline [â¦]
That. The respective commit was made on 2017-07-20, but feature
freeze was on 2017-07-18.
> The ldap.c file of 7.2.7 already contains the code for the
> refresh command.
While I can see the LDAP_EXOP_REFRESH constant, I can't find the
ldap_exop_refresh() function there.
------------------------------------------------------------------------
[2018-07-02 06:59:53] mcmic@php.net
I donât know why it was not added in 7.2, it might have been a mistake, or maybe I missed the
deadline I canât remember.
Iâm not sure we can add a feature in a 7.2 minor release now.
------------------------------------------------------------------------
[2018-06-29 11:08:21] requinix@php.net
I would expect that if it could be added to 7.2 then it would have been...
@mcmic?
------------------------------------------------------------------------
[2018-06-29 11:06:44] post at rolandgruber dot de
Thanks a lot for the info.
Can you also add it in 7.2? Seems HAVE_LDAP_REFRESH_S is missing here:
https://github.com/php/php-src/blob/PHP-7.2.7/ext/ldap/config.w32
AC_DEFINE('HAVE_LDAP_REFRESH_S', 1);
The ldap.c file of 7.2.7 already contains the code for the refresh command.
------------------------------------------------------------------------
[2018-06-29 11:03:47] requinix@php.net
ldap_exop_refresh is being added in 7.3.
https://github.com/php/php-src/commit/5e56d050e582d845e977377a1f8ba8deb15ab082
------------------------------------------------------------------------
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=76546
--
Edit this bug report at https://bugs.php.net/bug.php?id=76546&edit=1