From: Jens Lechtenboerger <lechten@wi.uni-muenster.de>
To: Filipp Gunbin <fgunbin@fastmail.fm>
Cc: 58605@debbugs.gnu.org, Juri Linkov <juri@linkov.net>
Subject: bug#58605: 29.0.50; Commit 2a2f5530fa2 breaks ldap-search-internal
Date: Sat, 22 Oct 2022 11:43:07 +0200 [thread overview]
Message-ID: <874jvwp51w.fsf@wi.uni-muenster.de> (raw)
In-Reply-To: <m2tu3x9phb.fsf@fastmail.fm> (Filipp Gunbin's message of "Fri, 21 Oct 2022 18:16:16 +0300")
[-- Attachment #1: Type: text/plain, Size: 1311 bytes --]
Hi there!
On 2022-10-21, Filipp Gunbin wrote:
> Oh, so this is not just a test case, but the code you actually use. Add
> "-LLL" to ldap-ldapsearch-args?
This was a test case. In my code [1], I use ldap-search, not the
internal function. Essentially, like this:
(let ((ldap-default-base "O=DFN-Verein,C=DE")
(ldap-ldapsearch-args '("-x" "-tt" "-H ldaps://ldap.pca.dfn.de"))
(mail "nix@example.org")
(host ""))
(ldap-search (concat "mail=" mail) host '("userCertificate") nil))
Previously, this just returned the userCertificate, with and without
-LLL. If no certificate existed, it returned nil.
Now, -LLL seems to be required, which I perceive to be a breaking
change. Can this be avoided? Alternatively, document it?
I confirm that with -LLL, my code works.
Without it, I get ((("search" " ") ("result" " "))) instead of nil.
Also, without -LLL and with an existing certificate, ("dn" " ") and
(("search" " ") ("result" " ")) are included in the result. Note
that I find ("dn" " ") misleading, as ldapsearch really does return
a dn value...
> BTW, that question mark in the regexp was added to fix M-x
> eudc-get-attribute-list, which was broken.
I was not aware of eudc, thanks!
Best wishes
Jens
[1] https://gitlab.com/lechten/defaultencrypt/-/blob/master/jl-smime.el
[-- Attachment #2: smime.p7s --]
[-- Type: application/pkcs7-signature, Size: 6187 bytes --]
next prev parent reply other threads:[~2022-10-22 9:43 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
2022-10-22 9:43 ` Jens Lechtenboerger [this message]
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
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=874jvwp51w.fsf@wi.uni-muenster.de \
--to=lechten@wi.uni-muenster.de \
--cc=58605@debbugs.gnu.org \
--cc=fgunbin@fastmail.fm \
--cc=juri@linkov.net \
/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 external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.