Horton

Re: SHOW ALL does not honor pg_read_all_settings membership

Michael Paquier <michael@paquier.xyz>

View thread View on postgr.es
From: Michael Paquier <michael@paquier.xyz>
To: Laurenz Albe <laurenz.albe@cybertec.at>
Cc: pgsql-hackers@postgresql.org, simon@2ndquadrant.com
Date: 2018-04-20T13:58:06Z
Lists: pgsql-hackers 
On Fri, Apr 20, 2018 at 01:21:46PM +0200, Laurenz Albe wrote:
> I agree; here is a patch for that.

Thanks for taking care of that as well this looks fine to me.  Note to
committer: this needs to be merged with the first patch actually fixing
the SHOW ALL issue.

All the four core callers of GetConfigOption() actually do not use
restrict_superuser set to true...  Modules may use this option, so of
course let's not remove it.
--
Michael

Commits

  1. Teach SHOW ALL to honor pg_read_all_settings membership

  2. Default monitoring roles