Re: [PATCH] Reload SSL certificates on SIGHUP - Mailing list pgsql-hackers

From Andreas Karlsson
Subject Re: [PATCH] Reload SSL certificates on SIGHUP
Date
Msg-id 6d45be21-637b-c802-75e7-1ea634ed4089@proxel.se
Whole thread Raw
In response to Re: [PATCH] Reload SSL certificates on SIGHUP  (Michael Paquier <michael.paquier@gmail.com>)
Responses Re: [PATCH] Reload SSL certificates on SIGHUP  (Michael Paquier <michael.paquier@gmail.com>)
List pgsql-hackers
On 12/04/2016 02:12 PM, Michael Paquier wrote:
> One last thing that I think is missing in this patch is for users the
> possibility to check via SQL if the SSL context is actually loaded or
> not. As the context is reloaded after all the new values are
> available, with the current patch users may see that ssl is set to on
> but no context is loaded. So why not adding for example a read-only
> parameter that maps with SSLLoaded?

The other three issues are easy to fix, but this one is a bit trickier. 
Do you mean that we should add another GUC here which is read-only? Does 
this have a precedent in the code?

Andreas



pgsql-hackers by date:

Previous
From: Julien Rouhaud
Date:
Subject: Re: Rename max_parallel_degree?
Next
From: Michael Paquier
Date:
Subject: Re: [PATCH] Reload SSL certificates on SIGHUP