From: Simon Josefsson <jas@extundo.com>
Subject: Re: Moving files from lisp/gnus/ to lisp/net/?
Date: Fri, 29 Oct 2004 23:25:47 +0200 [thread overview]
Message-ID: <iluoeilgqsk.fsf@latte.josefsson.org> (raw)
In-Reply-To: E1CNOR2-00066h-24@fencepost.gnu.org
Richard Stallman <rms@gnu.org> writes:
> Other applications typically ask the user whether they want to
> remember the password in memory. If read-passwd is changed to cache
> passwords (however, to use the cache, callers of read-passwd must be
> updated, to provide the "key" into the hash table), it could ask the
> user this. Opinions on this welcome.
>
> Having it always ask would be too annoying, I think. So that would
> need to be a new argument, which means the feature is no benefit
> unless we change the callers.
>
> That is the wrong thing to do at present. So I think we should put
> read-passwd back where it was and remove the new file for now.
I agree, password.el was only something that might go in when we
aren't in a feature freeze. I have backed out the change.
Since password.el will be used by Gnus CVS, our discussion haven't
been in vain, though, since I will incorporate some of the ideas
discussed here.
> Looking ahead to the future,
>
> > Why have both password-read and password-read-and-add?
> > Why not always add? Is the idea that for some purposes
> > it is ok to cache, but for others it is too risky?
>
> No, the reason was this: if the user entered an incorrect password, it
> should not be cached. If an incorrect password is cached, the code
> might infloop trying the incorrect password automatically over and
> over again. It was considered safer to first read the password, then
> try to use it, and if successful then it is cached.
>
> In that case, password-read-and-add makes no sense, right?
> Why add a shortcut that in best practice should not be used?
It was added later on, not by me. I suppose he who added it might
argue that it is useful, though. I tend to think that there should
only be one worked out interface. (I'm not saying the current API,
without p-r-a-a is the best interface, only that it would be better
with only one interface.)
> I'm not sure my argument is good, it may be simpler to always cache,
> and have the calling code invoke password-cache-remove whenever there
> is a password failure.
>
> That is a reasonable alternative, I guess, but then password-read
> should add the password and we should not have password-read-and-add
> as a separate entry point.
I think so too.
> But it doesn't make sense to discuss this without dealing with the
> question of whether caching of correct passwords is desirable.
Perhaps the experience with how password.el is used in Gnus CVS will
help in sorting things out.
Thanks.
next prev parent reply other threads:[~2004-10-29 21:25 UTC|newest]
Thread overview: 100+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-10-15 19:26 Moving files from lisp/gnus/ to lisp/net/? Simon Josefsson
2004-10-15 19:37 ` Simon Josefsson
2004-10-15 23:27 ` Miles Bader
2004-10-16 2:08 ` Ted Zlatanov
2004-10-16 13:52 ` Richard Stallman
2004-10-17 13:45 ` Simon Josefsson
2004-10-18 13:58 ` Richard Stallman
2004-10-24 12:24 ` Simon Josefsson
2004-10-25 13:13 ` Richard Stallman
2004-10-25 14:13 ` Simon Josefsson
2004-10-27 10:47 ` Richard Stallman
2004-10-27 19:08 ` Simon Josefsson
2004-10-29 4:32 ` Richard Stallman
2004-10-29 21:25 ` Simon Josefsson [this message]
2004-11-01 20:09 ` Ted Zlatanov
2004-11-01 20:31 ` Simon Josefsson
2004-11-02 18:30 ` Ted Zlatanov
2004-11-02 21:15 ` Stefan Monnier
2007-11-02 15:02 ` Richard Stallman
2007-11-05 10:16 ` Simon Josefsson
2007-11-06 8:38 ` Richard Stallman
2007-11-06 14:01 ` Daiki Ueno
2007-11-07 7:55 ` Richard Stallman
2007-11-08 0:46 ` Daiki Ueno
2007-11-08 17:40 ` Richard Stallman
[not found] ` <54a15d860711081633k364caf30r3b75464be8a060fa@mail.gmail.com>
2007-11-09 13:40 ` Fwd: " Daiki Ueno
[not found] ` <E1IqbtM-0003DR-P8@fencepost.gnu.org>
2007-11-10 4:53 ` Daiki Ueno
2007-11-10 7:07 ` Daiki Ueno
2007-11-10 20:03 ` Stefan Monnier
2007-11-11 0:04 ` Daiki Ueno
2007-11-11 19:33 ` Richard Stallman
2007-11-12 8:54 ` Daiki Ueno
2007-11-13 5:10 ` Richard Stallman
2007-11-14 3:45 ` Daiki Ueno
2007-11-14 5:44 ` Michael Albinus
2007-11-14 14:54 ` Stefan Monnier
2007-11-14 15:13 ` Michael Albinus
2007-11-14 17:30 ` Stefan Monnier
2007-11-15 3:08 ` Richard Stallman
2007-11-15 3:47 ` Stefan Monnier
2007-11-16 4:28 ` Richard Stallman
2007-11-15 0:23 ` Miles Bader
2007-11-15 3:08 ` Richard Stallman
2007-11-11 5:22 ` Richard Stallman
2007-11-10 19:53 ` Tom Tromey
2004-10-15 20:05 ` Miles Bader
2004-10-15 21:15 ` Simon Josefsson
2004-10-16 7:43 ` Thien-Thi Nguyen
2004-10-16 13:44 ` Miles Bader
2004-10-15 20:44 ` Stefan Monnier
2004-10-15 21:06 ` Simon Josefsson
2004-10-15 21:32 ` Lars Magne Ingebrigtsen
2004-10-15 21:47 ` Simon Josefsson
2004-10-15 21:52 ` Stefan Monnier
2004-10-18 1:04 ` Miles Bader
2004-10-18 13:59 ` Richard Stallman
2004-10-20 10:27 ` Lars Magne Ingebrigtsen
2004-10-20 10:50 ` Miles Bader
2004-10-20 11:22 ` Ralf Angeli
2004-10-20 11:39 ` Ralf Angeli
2004-10-21 1:09 ` Miles Bader
2004-10-20 11:32 ` Lars Magne Ingebrigtsen
2004-10-20 11:56 ` Karl Fogel
2004-10-20 12:38 ` David Kastrup
2004-10-20 13:06 ` Stefan Monnier
2004-10-20 12:34 ` David Kastrup
2004-10-21 1:28 ` Miles Bader
2004-10-21 5:03 ` Lars Magne Ingebrigtsen
2004-10-21 5:49 ` Miles Bader
2004-10-21 10:05 ` Kim F. Storm
2004-10-23 7:48 ` Lars Magne Ingebrigtsen
2004-10-25 1:02 ` Miles Bader
2004-10-25 10:14 ` Simon Josefsson
2004-10-25 17:46 ` Lars Magne Ingebrigtsen
2004-10-25 22:41 ` Miles Bader
2004-10-27 17:21 ` Lars Magne Ingebrigtsen
2004-10-28 2:21 ` Miles Bader
2004-10-26 7:28 ` Kai Grossjohann
2004-10-27 17:21 ` Lars Magne Ingebrigtsen
2004-10-16 13:52 ` Richard Stallman
2004-10-18 17:54 ` Ted Zlatanov
2004-10-18 21:36 ` Reiner Steib
2004-10-21 18:45 ` Ted Zlatanov
2004-10-22 0:25 ` Miles Bader
2004-10-24 12:13 ` Simon Josefsson
2004-10-24 12:20 ` Miles Bader
2004-10-24 13:41 ` Simon Josefsson
2004-10-24 22:18 ` Stefan
2004-10-25 10:08 ` Simon Josefsson
2004-10-25 11:55 ` Stefan
2004-10-25 13:07 ` Simon Josefsson
2004-10-26 9:05 ` Richard Stallman
2004-10-26 9:26 ` Simon Josefsson
2004-10-19 16:45 ` Richard Stallman
2004-10-21 18:45 ` Ted Zlatanov
2004-10-23 4:49 ` Richard Stallman
2004-10-24 12:59 ` Simon Josefsson
2004-10-24 13:05 ` Move sha1.el and hex-util.el from lisp/gnus/ to lisp/net/ (was: Re: Moving files from lisp/gnus/ to lisp/net/?) Simon Josefsson
2004-10-25 13:13 ` Richard Stallman
2004-10-25 14:23 ` Move sha1.el and hex-util.el from lisp/gnus/ to lisp/net/ Simon Josefsson
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
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=iluoeilgqsk.fsf@latte.josefsson.org \
--to=jas@extundo.com \
/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 public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).