From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Ted Zlatanov Newsgroups: gmane.emacs.bugs Subject: bug#45711: 27.1; Ambiguous account match in .authinfo still used by smtpmail Date: Sun, 10 Jan 2021 14:43:43 +0000 Organization: =?UTF-8?Q?=D0=A2=D0=B5=D0=BE=D0=B4=D0=BE=D1=80_?= =?UTF-8?Q?=D0=97=D0=BB=D0=B0=D1=82=D0=B0=D0=BD=D0=BE=D0=B2?= @ Cienfuegos Message-ID: References: <87lfd5osiu.fsf@bayne.id.au> <875z45adtm.fsf@bayne.id.au> Mime-Version: 1.0 Content-Type: text/plain Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="21287"; mail-complaints-to="usenet@ciao.gmane.io" User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (gnu/linux) Cc: 45711@debbugs.gnu.org To: Duncan Bayne Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Sun Jan 10 15:44:31 2021 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 1kybxF-0005Mj-SB for geb-bug-gnu-emacs@m.gmane-mx.org; Sun, 10 Jan 2021 15:44:29 +0100 Original-Received: from localhost ([::1]:42158 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1kybxE-0004Tf-Sl for geb-bug-gnu-emacs@m.gmane-mx.org; Sun, 10 Jan 2021 09:44:28 -0500 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:41128) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1kybwo-0004T2-Q9 for bug-gnu-emacs@gnu.org; Sun, 10 Jan 2021 09:44:02 -0500 Original-Received: from debbugs.gnu.org ([209.51.188.43]:42023) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1kybwo-0006Jt-FQ for bug-gnu-emacs@gnu.org; Sun, 10 Jan 2021 09:44:02 -0500 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1kybwo-0003QS-Dg for bug-gnu-emacs@gnu.org; Sun, 10 Jan 2021 09:44:02 -0500 X-Loop: help-debbugs@gnu.org Resent-From: Ted Zlatanov Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Sun, 10 Jan 2021 14:44:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 45711 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: fixed Original-Received: via spool by 45711-submit@debbugs.gnu.org id=B45711.161028983313135 (code B ref 45711); Sun, 10 Jan 2021 14:44:02 +0000 Original-Received: (at 45711) by debbugs.gnu.org; 10 Jan 2021 14:43:53 +0000 Original-Received: from localhost ([127.0.0.1]:53568 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1kybwf-0003Pm-HF for submit@debbugs.gnu.org; Sun, 10 Jan 2021 09:43:53 -0500 Original-Received: from mail-wr1-f49.google.com ([209.85.221.49]:46473) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1kybwe-0003PR-2I for 45711@debbugs.gnu.org; Sun, 10 Jan 2021 09:43:52 -0500 Original-Received: by mail-wr1-f49.google.com with SMTP id d13so13691048wrc.13 for <45711@debbugs.gnu.org>; Sun, 10 Jan 2021 06:43:51 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=lifelogs.com; s=google; h=from:to:cc:subject:organization:references:date:in-reply-to :message-id:user-agent:mime-version; bh=AD7wSuTfdMvy3K8UNqXe0M+1cI/ufD4UrQbclefAOdg=; b=Mke533IpSYFP4e5gAtWV4EOdrywrupRE1l9RhAN/QenGfveKH7L4CD8DCXA+bliEli dD5bcZKIaVE1oAyS02VUDGf6u++O5zQfWuQzwpzUelUlt/bZ6oZ9mlpOaAA1X0htAHYT KKIHP6iSzQWZ474oQobCR14A2iZyAPc7k4m6E= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:organization:references:date :in-reply-to:message-id:user-agent:mime-version; bh=AD7wSuTfdMvy3K8UNqXe0M+1cI/ufD4UrQbclefAOdg=; b=MTtuT/dti/EEuqooRIlipFAiOkGQ92eYB4DnsCEh6wJvQONRPNyc+xJ1gMX63gOa/W nwA/mw6f3cR238Z3TIBxRbKRmQ8q6hhL30BL+zynhJgY4wDzvy6NvoFWEFNU661HvC1i wL/QXghhSP5KMSVbStnMlpSOBQAog6VRwrBUWhda/pwUGq1jDUkSiR51O9pPE/p8ZodB Fg5ND8whGeaX54lw2/HQrgGjHHpoeDBiV8+VqgSehxj+rsGCejHxm9kg3uozfuY1l9aF ELG/EbTIUydzkLiomvHOAyu4mcFIB1q0pP3Y6eB08fDSi6nmzxcPzV0oBrkS8aBTivdG 5cMw== X-Gm-Message-State: AOAM532ghezYnsAje4g8kEc9T+vWPsBhxh7H11kVz6ZNXUoIrEBSzsHl bd0ulSEivc/gasynpIyWfnj3qqgC7IiNWg== X-Google-Smtp-Source: ABdhPJzHv6lGesIAlABhYyr6Z6wR4Pa+lC5pGiy9OJsWycZ11I8oq1tHAsCBD9+4G7oyz+mdXurPDA== X-Received: by 2002:adf:e452:: with SMTP id t18mr11899721wrm.177.1610289825638; Sun, 10 Jan 2021 06:43:45 -0800 (PST) Original-Received: from flea ([176.248.35.185]) by smtp.gmail.com with ESMTPSA id m8sm19757769wmc.27.2021.01.10.06.43.44 (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Sun, 10 Jan 2021 06:43:44 -0800 (PST) X-Face: bd.DQ~'29fIs`T_%O%C\g%6jW)yi[zuz6; d4V0`@y-~$#3P_Ng{@m+e4o<4P'#(_GJQ%TT= D}[Ep*b!\e,fBZ'j_+#"Ps?s2!4H2-Y"sx" In-Reply-To: <875z45adtm.fsf@bayne.id.au> (Duncan Bayne's message of "Sun, 10 Jan 2021 10:30:13 +1100") 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" Xref: news.gmane.io gmane.emacs.bugs:197618 Archived-At: On Sun, 10 Jan 2021 10:30:13 +1100 Duncan Bayne wrote: DB> Ted Zlatanov writes: DB> I think it's the right thing for auth-source-search to return a list of DB> matches, but I think it's a bug for smtpmail-try-auth-methods to just DB> quietly use the first match in that case. OK. ... DB> What it seems to be doing with (car (... :max 1) is to simply take the DB> first match and use that. DB> What I *think* it should do is either a) error out in the case of an DB> ambiguous match, or b) prompt the use to ask which of the various DB> matches it should use. I see. Your approach makes sense, I agree, but it will be surprising to people who intentionally have multiple matches. Maybe we need to search with :max 2. There, if the :max 2 search returns two equally specific entries, that's probably an error, or at least a strong warning (sit-for 3 seconds or something). But if the :max 2 search returns a less specific (more general) entry in the second slot, then that's probably what the user intended. What do you think? DB> Meta: this is my first Emacs bug report, and I'm planning to write a DB> patch to address it. Not sure whether the discussion of whether it's a DB> bug, and if so, what the appropriate fix is, belongs here or on the DB> emacs-devel list. Happy to take the conversation there if that's DB> protocol. No, here is perfect. If you provide a patch I can apply it (unless it's non-trivial, in which case you'll need to sign the contributor papers). Ted