From: | Stephen Frost <sfrost(at)snowman(dot)net> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Andrew Dunstan <andrew(at)dunslane(dot)net>, Itagaki Takahiro <itagaki(dot)takahiro(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Add support for logging the current role |
Date: | 2011-02-17 16:18:32 |
Message-ID: | 20110217161832.GA4116@tamriel.snowman.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
* Tom Lane (tgl(at)sss(dot)pgh(dot)pa(dot)us) wrote:
> Well, that just doesn't seem useful to me in the real world. If I were
> using this, I would expect it to emit a real user name that matches the
> currently applied permissions checking. All the time.
I wouldn't have ever thought to use %U w/o %u, to be honest. Unless I'm
missing something though, this change would just be emitting what
show_session_authorization() returns when show_role() returns 'none'.
That's certainly fine by me.
> "show role" does
> what it does because the SQL standard says so, not because anybody
> outside the standards committee thinks that's a sane definition.
Guess it actually makes some sense to me.
Thanks,
Stephen
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2011-02-17 16:22:53 | Re: Debian readline/libedit breakage |
Previous Message | Kevin Grittner | 2011-02-17 16:18:23 | Re: Re: [COMMITTERS] pgsql: Fix blatantly uninitialized variable in recent commit. |