GNU bug report logs - #58605
29.0.50; Commit 2a2f5530fa2 breaks ldap-search-internal

Previous Next

Full log

Message #20 received at 58605 <at> debbugs.gnu.org (full text, mbox):

From: Filipp Gunbin <fgunbin <at> fastmail.fm>
To: Jens Lechtenboerger <lechten <at> wi.uni-muenster.de>
Cc: 58605 <at> debbugs.gnu.org, Juri Linkov <juri <at> linkov.net>
Subject: Re: bug#58605: 29.0.50; Commit 2a2f5530fa2 breaks ldap-search-internal
Date: Fri, 21 Oct 2022 18:16:16 +0300

Hi,

On 21/10/2022 08:25 +0200, Jens Lechtenboerger wrote:

> Hi all,
>
> On 2022-10-20, Filipp Gunbin wrote:
>
>> Jens, did you by chance modify ldap-ldapsearch-args?  Its default value
>> contains "-LLL" which suppresses noise like "search: 2" that you see.
>
> yes, I change that variable as indicated in my bug report:
>
>>> (let ((ldap-default-base "O=DFN-Verein,C=DE")
>>>       (ldap-ldapsearch-args '("-x" "-tt" "-H ldaps://ldap.pca.dfn.de"))
>>>       (search-plist '(host "" filter "mail=x <at> example.org" attributes ("userCertificate") attrsonly nil withdn nil)))
>>>   (ldap-search-internal search-plist))
>
> Previously, this worked.

Oh, so this is not just a test case, but the code you actually use.  Add
"-LLL" to ldap-ldapsearch-args?

>> I've now pushed the fix to master, could you please try?
>
> I do not get an error any more, which is good, but also not nil,
> which might break prior code: ((("search" " ") ("result" " ")))

Let's see.

> Also, the format of return values for existing certificates seems to
> have changed.  I hope to find time during the weekend to have a
> closer look...

Yeah, please report if you see problems.

BTW, that question mark in the regexp was added to fix M-x
eudc-get-attribute-list, which was broken.

Filipp

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.