Re: Again, the requirement of password of dlgUser.cpp

From: "Dave Page" <dpage(at)vale-housing(dot)co(dot)uk>
To: "Hiroshi Saito" <saito(at)inetrt(dot)skcapi(dot)co(dot)jp>, "Andreas Pflug" <Andreas(dot)Pflug(at)web(dot)de>
Cc: <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: Again, the requirement of password of dlgUser.cpp
Date: 2005-03-16 08:40:08
Message-ID: E7F85A1B5FF8D44C8A1AF6885BC9A0E472BB8B@ratbert.vale-housing.co.uk
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

> -----Original Message-----
> From: Hiroshi Saito [mailto:saito(at)inetrt(dot)skcapi(dot)co(dot)jp]
> Sent: 16 March 2005 00:23
> To: Andreas Pflug
> Cc: pgadmin-hackers(at)postgresql(dot)org; Dave Page
> Subject: Again, the requirement of password of dlgUser.cpp
>
> Hi Andreas.
>
> I have this requirement said by a user many times.:-(
> We should hide password which it is inputted from.
>
> --- src/ui/common/dlgUser.xrc.orig Tue Mar 15 10:25:19 2005
> +++ src/ui/common/dlgUser.xrc Tue Mar 15 10:26:02 2005
> @@ -25,6 +25,7 @@
> <object class="wxStaticText" name="stPasswd">
> <label>Password</label>
> <pos>5,37d</pos>
> + <style>wxTE_PASSWORD</style>
> </object>
> <object class="wxTextCtrl" name="txtPasswd">
> <pos>70,35d</pos>
>
> You said that it was meaningless because there was SQL which did
> reverse in the SQL Frame. but, meaning is different from it.
> user will
> make it "password_encryption = true" of postgresql.conf. Then, when
> it is inputted, it is made to be disappointed.:-(
> This is strongly required.

OK, I'll agree with that - the password is not shown in the SQL pane by
default these days.

However - we need to improve your patch Hiroshi:

1) You should add the style to txtPasswd, not stPasswd :-)

2) Because it cannot be seen anymore, it should be confirmed - so, can
you add a 'Confirm Password' textbox with the appropriate code as well
please?

Thanks, Dave

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message cvs 2005-03-16 08:42:43 CVS Commit by dpage: Updated translation from Slawomir Sudnik
Previous Message Hiroshi Saito 2005-03-16 00:23:16 Again, the requirement of password of dlgUser.cpp