Konstantin Ryabitsev <[email protected]> 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 [email protected] > 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 <[email protected]> --- 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<LOCATION> is a source of messages: a directory (Maildir), a file, or a URL (C<imap://>, C<imaps://>, C<nntp://>, or C<nntps://>). URLs requiring -authentication must use L<netrc(5)> and/or L<git-credential(1)> to +authentication use L<git-credential(1)> to fill in the username and password. For a regular file, the location must have a C<E<lt>formatE<gt>:> 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<lei(1)>. C<LOCATION> is a source of messages: a directory (Maildir), a file, or a URL (C<imap://>, C<imaps://>, C<nntp://>, or C<nntps://>). URLs requiring -authentication must use L<netrc(5)> and/or L<git-credential(1)> to +authentication use L<git-credential(1)> to fill in the username and password. For a regular file, the location must have a C<E<lt>formatE<gt>:> 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</tmp/results-Maildir>, C<imaps://[email protected]/INBOX.test>, or C<mboxcl2:/tmp/results-mboxcl2>). The prefix may be a supported protocol: C<imap://> or C<imaps://>. URLs requiring -authentication must use L<netrc(5)> and/or L<git-credential(1)> to +authentication use L<git-credential(1)> to fill in the username and password. A prefix can specify the format of the output: C<maildir>, 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<imap://> and C<imaps://> URLs: This may be specified multiple times to combine several mailboxes into a single public-inbox. URLs requiring authentication -will require L<netrc(5)> and/or L<git-credential(1)> to fill +will require L<netrc(5)> and/or L<git-credential(1)> (preferred) to fill in the username and password. Default: none -- unsubscribe: one-click, see List-Unsubscribe header archive: https://public-inbox.org/meta/
