[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