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

[William Jon McCann]

On 9/12/07, Simo Sorce <ssorce at redhat.com> wrote:
> 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 ...

Oh.  That wasn't obvious to me.  In any case it is probably better to
default to "yes" for the suggested realm name.

Jon