Re: Audit of logout - Mailing list pgsql-hackers

From Amit Kapila
Subject Re: Audit of logout
Date
Msg-id CAA4eK1LmmENBP39a=ib20YPfBb-guu3LWNOPFEhoBNhoOyLjgw@mail.gmail.com
Whole thread Raw
In response to Re: Audit of logout  (Fujii Masao <masao.fujii@gmail.com>)
Responses Re: Audit of logout
List pgsql-hackers
On Wed, Aug 27, 2014 at 5:19 PM, Fujii Masao <masao.fujii@gmail.com> wrote:
>
> On Sat, Aug 23, 2014 at 3:44 PM, Amit Kapila <amit.kapila16@gmail.com> wrote:
> > On Tue, Aug 5, 2014 at 8:04 PM, Fujii Masao <masao.fujii@gmail.com> wrote:
> > Changing PGC_SU_BACKEND parameter (log_connections) is
> > visible even with a non-super user client due to above code.
> > Shouldn't it be only visible for super-user logins?
> >
> > Simple steps to reproduce the problem:
> > a. start Server (default configuration)
> > b. connect with superuser
> > c. change in log_connections to on in postgresql.conf
> > d. perform select pg_reload_conf();
> > e. connect with non-super-user
> > f.  show log_connections;  --This step shows the value as on,
> >                                        --whereas I think it should have been
> > off
>
> In this case, log_connections is changed in postgresql.conf and it's
> reloaded, so ISTM that it's natural that even non-superuser sees the
> changed value. No? Maybe I'm missing something.

Yeah, you are right.

With the latest patch, I am getting one regression failure on windows.
Attached is the regression diff.

Can we improve this line a bit?
!  * BACKEND options are the same as SU_BACKEND ones, but they can
BACKEND options can be set same as SU_BACKEND ones, ......


With Regards,
Amit Kapila.
EnterpriseDB: http://www.enterprisedb.com
Attachment

pgsql-hackers by date:

Previous
From: Magnus Hagander
Date:
Subject: Re: re-reading SSL certificates during server reload
Next
From: Andreas Karlsson
Date:
Subject: Re: INSERT ... ON CONFLICT {UPDATE | IGNORE}