[Freeipa-devel] [PATCH] don't echo passwords

William Jon McCann mccann at jhu.edu
Wed Sep 12 17:37:28 UTC 2007


On 9/12/07, Simo Sorce <ssorce at redhat.com> wrote:
> On Wed, 2007-09-12 at 11:27 -0400, William Jon McCann wrote:
> >
> > A patch for the password echo is attached.  I'll send separate patches
> > for the other issues or any more that I encounter.
>
> Pushed and added also a patch to ask for confirmation.

Thanks.

> Couldn't find y/N questions w/o the [y/N] hint, are you looking at the
> mercurial repository or the code in one of the drops/milestones?

Sorry I wasn't very clear.  I mean that there is no guidance as to
what the default is.  From looking at the code it seems that the
default answer is 'no'.  I would suggest something like:

Do you want to use DOMAIN as the realm name? [yes] :

So that a return accepts the default which in this case is "yes".
Also probably good to handle case insensitve "y" and "yes" as
affirmative.

A minor thing but a lot of people including myself will just choose
the defaults the first time they test the software...

Jon




More information about the Freeipa-devel mailing list