From: Filipp Gunbin <fgunbin@fastmail.fm>
To: Jens Lechtenboerger <lechten@wi.uni-muenster.de>
Cc: 58605@debbugs.gnu.org, Juri Linkov <juri@linkov.net>
Subject: bug#58605: 29.0.50; Commit 2a2f5530fa2 breaks ldap-search-internal
Date: Fri, 21 Oct 2022 18:16:16 +0300 [thread overview]
Message-ID: <m2tu3x9phb.fsf@fastmail.fm> (raw)
In-Reply-To: <87ilkd7kww.fsf@wi.uni-muenster.de> (Jens Lechtenboerger's message of "Fri, 21 Oct 2022 08:25:35 +0200")
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@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
next prev parent reply other threads:[~2022-10-21 15:16 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-18 6:05 bug#58605: 29.0.50; Commit 2a2f5530fa2 breaks ldap-search-internal Jens Lechtenboerger
2022-10-18 14:17 ` Eli Zaretskii
2022-10-20 8:12 ` Juri Linkov
2022-10-20 17:58 ` Filipp Gunbin
2022-10-21 6:25 ` Jens Lechtenboerger
2022-10-21 15:16 ` Filipp Gunbin [this message]
2022-10-22 9:43 ` Jens Lechtenboerger
2022-10-24 16:06 ` Filipp Gunbin
2022-10-25 5:55 ` Jens Lechtenboerger
2022-10-25 13:12 ` Filipp Gunbin
2022-10-25 13:42 ` Jens Lechtenboerger
2022-10-25 15:07 ` Filipp Gunbin
2022-10-26 6:07 ` Jens Lechtenboerger
2022-10-26 13:20 ` Filipp Gunbin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=m2tu3x9phb.fsf@fastmail.fm \
--to=fgunbin@fastmail.fm \
--cc=58605@debbugs.gnu.org \
--cc=juri@linkov.net \
--cc=lechten@wi.uni-muenster.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).