Skip to content

Commit 4391c73

Browse files
Merge pull request #45357 from nextcloud/fix/hide-ldap-password-as-sensitive
fix: add ldap_exop_passwd function to sensitive value
2 parents ee8a97b + 7c91ab1 commit 4391c73

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

apps/user_ldap/lib/LDAP.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -319,7 +319,7 @@ protected function invokeLDAPMethod(string $func, ...$arguments) {
319319

320320
private function preFunctionCall(string $functionName, array $args): void {
321321
$this->curArgs = $args;
322-
if(strcasecmp($functionName, 'ldap_bind') === 0) {
322+
if(strcasecmp($functionName, 'ldap_bind') === 0 || strcasecmp($functionName, 'ldap_exop_passwd') === 0) {
323323
// The arguments are not key value pairs
324324
// \OCA\User_LDAP\LDAP::bind passes 3 arguments, the 3rd being the pw
325325
// Remove it via direct array access for now, although a better solution could be found mebbe?

0 commit comments

Comments
 (0)