From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Filipp Gunbin Newsgroups: gmane.emacs.bugs Subject: bug#64160: master; Implement various enhancements in ldap.el and EUDC Date: Tue, 20 Jun 2023 17:11:59 +0300 Message-ID: References: <83cz1usk0i.fsf@gnu.org> <83legiqy3x.fsf@gnu.org> <83h6r6qwxh.fsf@gnu.org> <704ffb16-a822-8fed-6d63-4d9fc9a2b16e@vodafonemail.de> <05a07735-fbed-e8a2-511a-3996e1e241ea@vodafonemail.de> Mime-Version: 1.0 Content-Type: text/plain Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="39421"; mail-complaints-to="usenet@ciao.gmane.io" User-Agent: Gnus/5.13 (Gnus v5.13) Cc: 64160@debbugs.gnu.org To: Jens Schmidt Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Tue Jun 20 16:14:21 2023 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane-mx.org Original-Received: from lists.gnu.org ([209.51.188.17]) by ciao.gmane.io with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1qBc7e-0009uO-4R for geb-bug-gnu-emacs@m.gmane-mx.org; Tue, 20 Jun 2023 16:14:20 +0200 Original-Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1qBc7Q-00054K-Eq; Tue, 20 Jun 2023 10:14:04 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1qBc7P-00054C-36 for bug-gnu-emacs@gnu.org; Tue, 20 Jun 2023 10:14:03 -0400 Original-Received: from debbugs.gnu.org ([209.51.188.43]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1qBc7O-000138-QT for bug-gnu-emacs@gnu.org; Tue, 20 Jun 2023 10:14:02 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1qBc7O-0001M3-93 for bug-gnu-emacs@gnu.org; Tue, 20 Jun 2023 10:14:02 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Filipp Gunbin Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Tue, 20 Jun 2023 14:14:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 64160 X-GNU-PR-Package: emacs Original-Received: via spool by 64160-submit@debbugs.gnu.org id=B64160.16872704115158 (code B ref 64160); Tue, 20 Jun 2023 14:14:02 +0000 Original-Received: (at 64160) by debbugs.gnu.org; 20 Jun 2023 14:13:31 +0000 Original-Received: from localhost ([127.0.0.1]:59498 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1qBc6t-0001L7-6o for submit@debbugs.gnu.org; Tue, 20 Jun 2023 10:13:31 -0400 Original-Received: from wout5-smtp.messagingengine.com ([64.147.123.21]:44401) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1qBc6p-0001Kl-Tg for 64160@debbugs.gnu.org; Tue, 20 Jun 2023 10:13:29 -0400 Original-Received: from compute2.internal (compute2.nyi.internal [10.202.2.46]) by mailout.west.internal (Postfix) with ESMTP id E0BDB32009AE; Tue, 20 Jun 2023 10:13:19 -0400 (EDT) Original-Received: from mailfrontend2 ([10.202.2.163]) by compute2.internal (MEProxy); Tue, 20 Jun 2023 10:13:20 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=fastmail.fm; h= cc:cc:content-type:content-type:date:date:from:from:in-reply-to :in-reply-to:message-id:mime-version:references:reply-to:sender :subject:subject:to:to; s=fm2; t=1687270399; x=1687356799; bh=cY xxdgG/RWoKsYGwA1h15MU13JJHHQdnl1SS5rY39MQ=; b=oK/i8AkMMhM0EKyoHm ULGWfTpdBZxxvQsl0sDWCGbxWPdyUhso2GPl81DhFj4Z4IQmphFu9bNnNQkSWdbi S16Uzmpa/Nq/ALbb09LfUSH7Z07mGJEDJW3rc81JGWCpcNSqLaOo01HlSvXiTq/3 5oskmGDeKj9hM1MEn3o6YGJ1V6Rg0i25rK42Fajj+Rd/jHJGNo1O8xFomlTB2pcd wRfcCHjRynWrLsNYjlk41CPFQkAVOpaPptna2LaUUWYMVFjDbIqEQI0QvZQZaN56 1vSWm8JVtUMEPBZAsTvLHmX8PCHi9ac+4bKeCWLcExnJpWgs9I7xc5LJpesnzRcW uL1w== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-type:content-type:date:date :feedback-id:feedback-id:from:from:in-reply-to:in-reply-to :message-id:mime-version:references:reply-to:sender:subject :subject:to:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm2; t=1687270399; x=1687356799; bh=cYxxdgG/RWoKs YGwA1h15MU13JJHHQdnl1SS5rY39MQ=; b=ov1/4KNnqqzyhjiK6AqRjieQbZ+wf mckah74DJ8uN7v3zScvo4F9ntaL2IaGS77FJfB0+xQUgPYmIfQ5GBrfWbTSVhKDp DYNjkne3IhCV64eJYyKJX3IQJDPXH+lI9Gds7QzZlXLfexuzyEKh1UuGzRynLvPW q/P+v6g0Ib6xRh7XaaFOSZMvA5ZZq5p9aImVA7GY5ymUv3MRriUGsdQ9he5aTZNJ Egtv0VkL3T4tSI0aywzJTZ7GDItbEutCV+c4nxiIjOJYG1jXGv/kgElBsasCkZ+z ffv6wP1MZEwN6GOcji32D/LtcHVHKKGIaThQq400G+Iy1DE7kzZDfuw8w== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvhedrgeefhedghedvucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucenucfjughrpefhvfevufgjfhgffffkgggtsehttd ertddtredtnecuhfhrohhmpefhihhlihhpphcuifhunhgsihhnuceofhhguhhnsghinhes fhgrshhtmhgrihhlrdhfmheqnecuggftrfgrthhtvghrnhepfffhjeevieeiiedutefghe ffleeiveeffeeftdekleehledufeeuueduheevieffnecuvehluhhsthgvrhfuihiivgep tdenucfrrghrrghmpehmrghilhhfrhhomhepfhhguhhnsghinhesfhgrshhtmhgrihhlrd hfmh X-ME-Proxy: Feedback-ID: id48c41f1:Fastmail Original-Received: by mail.messagingengine.com (Postfix) with ESMTPA; Tue, 20 Jun 2023 10:13:18 -0400 (EDT) In-Reply-To: <05a07735-fbed-e8a2-511a-3996e1e241ea@vodafonemail.de> (Jens Schmidt's message of "Mon, 19 Jun 2023 23:16:58 +0200") X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Original-Sender: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Xref: news.gmane.io gmane.emacs.bugs:263764 Archived-At: Hi Jens, On 19/06/2023 23:16 +0200, Jens Schmidt wrote: > On 2023-06-19 17:13, Filipp Gunbin wrote: > >> Ok, I now see that it's about different things, let's move back this >> discussion to 64089. > > Problem is that the result of my plans in bug#64160 would be more or > less a complete rewrite of `ldap-search-internal'. For example, our > LDAP server returns underscores in its attribute names, which I would > have accounted for ... in a clean and configurable way and strictly > backward compatible, of course. I'd like to optionally have the record > attributes sorted in the order they came in from EUDC. I'd like to > optionally get rid of temporary files, since Emacs should be more than > capable to parse the base64-encoded attributes instead. And so on and > so forth ... there is really much room for improvement in that > function. > > Now from your previous comments I got the impression that we have a > different approach on coding and that you are somewhat opposed to changes. > > So I'd rather leave the changes in my private repository and save us > both time and energy. > > Feel free to close this bug. No, that's not my position to be opposed to changes. Patches are always welcome. I just keep in mind that who knows which ldap servers are deployed out there (and clients, but I guess OpenLDAP is de facto standart now), and we should be careful when changing things. Also, it's worth noting that seemingly few people use ldap.el, so we won't notice the break immediately, but rather when Emacs release comes out. Let's discuss each issue separately, as they arise, and see what comes out. From what you described - that certainly doesn't look like a "complete rewrite". I'd just like to note that perhaps I will be opposed to getting rid of temporary files. process-connection-type is not forced to pipe, so you may have tty sometimes, and receiving binary data over it is not reliable. Thanks, Filipp