Re: [HACKERS] Re: [COMMITTERS] 'pgsql/src/bin/initdb initdb.sh'

On Tue, 24 Feb 1998, Jan Wieck wrote:

> CREATE TABLE xpg_usr (
> usename name,
> usesysid int4,
> usecreatedb bool,
> usetrace bool,
> usesuper bool,
> usecatupd bool,
> passwd text,
> valuntil abstime);
>
> UPDATE pg_class SET relname = 'pg_usr'
> WHERE relname = 'xpg_usr';
>
> CREATE RULE _RETpg_usr AS ON SELECT TO pg_usr
> DO INSTEAD
> SELECT usename, usesysid, usecreatedb,
> usetrace, usesuper, usecatupd,
> '********'::text as passwd, valuntil
> FROM pg_user;
>
> REVOKE ALL ON pg_user FROM public;

Okay, just so that I don't mess things up in the translation...do
you want to send me an appropriate patch for this?

> > But as it was done in most UN*X's, could we rename the
> > pg_user containing the password into pg_shadow and then
> > create a view pg_user that just stars out the password field?
> > This way no existing application code (not even the JDBC
> > etc.) needs any changes, except for the createuser etc.
> > tools that always get installed with the new release.
>
> Still vote for this. And as soon as we finally choose one
> name for the public pg_user view we must fix createuser.sh,
> createdb.sh and so on to make their checks on the public
> accessible view so they still print the proper error messages
> instead of

No arguments here...can you include this as part of your patch
too?