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

[Simo Sorce]

On Wed, 2007-09-12 at 13:37 -0400, William Jon McCann wrote:
> 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...

The default is the one presented as Capital:
[y/N] defaults to N
[Y/n] defaults to Y

This is what we used in other software too ...

Simo.