| 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: | Whole Thread | Raw Message | 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
| 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 |
| 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 |
| 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 |