@raphael.badawi

Account created on 26 October 2022, almost 2 years ago
#

Recent comments

Beware, the patch ldap-php-8-compatibility-3302242-7 breaks LDAP pagination.

Instead of having:
```
ldap_search($this->connection, $ldap_query_params['base_dn'], $ldap_query_params['filter'], array("*"), 0, $this->searchPageSize, -1, LDAP_DEREF_NEVER, $servercontrols);
$result = $this->ldapQuery($ldap_query_params['scope'], $ldap_query_params);
```
we should just have :
```
$result = ldap_search($this->connection, $ldap_query_params['base_dn'], $ldap_query_params['filter'], array("*"), 0, $this->searchPageSize, -1, LDAP_DEREF_NEVER, $servercontrols);
```

The ldap_servers_php_supports_pagination has not been updated either. If patched as follows, it works again:
```
function ldap_servers_php_supports_pagination() {
if (version_compare(PHP_VERSION, '8.0.0') >= 0) {
return TRUE;
}
return (boolean)(function_exists('ldap_control_paged_result_response') && function_exists('ldap_control_paged_result'));
}
```

Despite these pagination issues, everything else works great with this patch.

Production build 0.71.5 2024