From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: X-Spam-Status: No, score=-4.0 required=3.0 tests=ALL_TRUSTED,BAYES_00 shortcircuit=no autolearn=ham autolearn_force=no version=3.4.2 Received: from localhost (dcvr.yhbt.net [127.0.0.1]) by dcvr.yhbt.net (Postfix) with ESMTP id AA5F31F8C6; Tue, 14 Sep 2021 20:12:16 +0000 (UTC) Date: Tue, 14 Sep 2021 20:12:16 +0000 From: Eric Wong To: Konstantin Ryabitsev Cc: meta@public-inbox.org Subject: Re: [PATCH] uri_imap: handle '/' as an IMAP hierarchy separator Message-ID: <20210914201216.GA29905@dcvr> References: <20210914175025.eq7s2shkc323itaf@meerkat.local> <20210914193528.29676-1-e@80x24.org> <20210914195510.kozlchcwvgrng5uc@meerkat.local> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20210914195510.kozlchcwvgrng5uc@meerkat.local> List-Id: Konstantin Ryabitsev wrote: > On Tue, Sep 14, 2021 at 07:35:28PM +0000, Eric Wong wrote: > > Oops, I think the patch below should fix it. > > Yep, that worked. Thanks! Cool. > Good to know, thanks. Quick follow-up -- documentation says that .netrc should > work, but I've found that even though I have the following entries in > ~/.netrc, I still get prompted for credentials: > > machine imap.migadu.com > login konstantin.ryabitsev@linux.dev > password [...] > > The credential helper works after the initial "lei up" but I'm curious why > .netrc isn't happy. Not a huge deal, seeing as that requires storing passwords > in plaintext. Ah, I forgot to update the docs again :x My main concern with .netrc was actually inadvertantly sending FTP auth info to an IMAP server just because they share the same host. Not sure if plaintext is a real problem on encrypted block devices/filesystems. Ordinary users can't mlock(2) to prevent in-memory passwords from hitting swap (thus I always use encrypted swap). -----------8<--------- Subject: [PATCH] doc: update authentication notes for lei ~/.netrc isn't used by default any more, and I'm not sure it's worthwhile to document the --netrc switch since it's rare for non-FTP clients to support. Followup-to: 9d11ed460ce113dd ("lei: do not read ~/.netrc by default") Reported-by: Konstantin Ryabitsev --- Documentation/lei-convert.pod | 2 +- Documentation/lei-import.pod | 2 +- Documentation/lei-q.pod | 2 +- Documentation/public-inbox-watch.pod | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/Documentation/lei-convert.pod b/Documentation/lei-convert.pod index 7f372327..750ba54f 100644 --- a/Documentation/lei-convert.pod +++ b/Documentation/lei-convert.pod @@ -13,7 +13,7 @@ lei convert [OPTIONS] (--stdin|-) Convert messages to another format. C is a source of messages: a directory (Maildir), a file, or a URL (C, C, C, or C). URLs requiring -authentication must use L and/or L to +authentication use L to fill in the username and password. For a regular file, the location must have a CformatE:> diff --git a/Documentation/lei-import.pod b/Documentation/lei-import.pod index c29a085b..9322dfa8 100644 --- a/Documentation/lei-import.pod +++ b/Documentation/lei-import.pod @@ -13,7 +13,7 @@ lei import [OPTIONS] (--stdin|-) Import messages into the local storage of L. C is a source of messages: a directory (Maildir), a file, or a URL (C, C, C, or C). URLs requiring -authentication must use L and/or L to +authentication use L to fill in the username and password. For a regular file, the location must have a CformatE:> diff --git a/Documentation/lei-q.pod b/Documentation/lei-q.pod index 1d9e66cd..2823ced8 100644 --- a/Documentation/lei-q.pod +++ b/Documentation/lei-q.pod @@ -43,7 +43,7 @@ Destination for results (e.g., C, C, or C). The prefix may be a supported protocol: C or C. URLs requiring -authentication must use L and/or L to +authentication use L to fill in the username and password. A prefix can specify the format of the output: C, diff --git a/Documentation/public-inbox-watch.pod b/Documentation/public-inbox-watch.pod index 5fa966be..e8f97c80 100644 --- a/Documentation/public-inbox-watch.pod +++ b/Documentation/public-inbox-watch.pod @@ -82,7 +82,7 @@ C and C URLs: This may be specified multiple times to combine several mailboxes into a single public-inbox. URLs requiring authentication -will require L and/or L to fill +will require L and/or L (preferred) to fill in the username and password. Default: none