Re: Bug #928: server_min_messages (log_min_messages in CVS) have PGC_USERSET GucContext

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: syatskevich(at)n21lab(dot)gosniias(dot)msk(dot)ru, pgsql-bugs(at)postgresql(dot)org
Cc: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
Subject: Re: Bug #928: server_min_messages (log_min_messages in CVS) have PGC_USERSET GucContext
Date: 2003-03-30 23:23:50
Message-ID: 12856.1049066630@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-hackers pgsql-patches

pgsql-bugs(at)postgresql(dot)org writes:
> In src/backend/utils/misc/guc.c "server_min_messages"
> ("log_min_messages" in CVS)configuration option have PGC_USERSET
> GucContext. I think that it is not good idea that user can
> change server log details level. I suggest change GucContext in
> this case on PGC_SIGHUP.

PGC_SUSET would be appropriate if we think that there's really a security
issue here. But ISTM this was already considered when the present setup
was designed, and we deliberately chose USERSET. Bruce, do you remember
what the reasoning was?

regards, tom lane

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Bruce Momjian 2003-03-31 00:20:29 Re: Bug #928: server_min_messages (log_min_messages in CVS) have
Previous Message pgsql-bugs 2003-03-30 23:06:17 Bug #928: server_min_messages (log_min_messages in CVS) have PGC_USERSET GucContext

Browse pgsql-hackers by date

  From Date Subject
Next Message Bruce Momjian 2003-03-31 00:20:29 Re: Bug #928: server_min_messages (log_min_messages in CVS) have
Previous Message pgsql-bugs 2003-03-30 23:06:17 Bug #928: server_min_messages (log_min_messages in CVS) have PGC_USERSET GucContext

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2003-03-31 00:20:29 Re: Bug #928: server_min_messages (log_min_messages in CVS) have
Previous Message pgsql-bugs 2003-03-30 23:06:17 Bug #928: server_min_messages (log_min_messages in CVS) have PGC_USERSET GucContext