Re: [PATCH] add ssl_protocols configuration option - Mailing list pgsql-hackers

From Alex Shulgin
Subject Re: [PATCH] add ssl_protocols configuration option
Date
Msg-id 87y4q9xalf.fsf@commandprompt.com
Whole thread Raw
In response to Re: [PATCH] add ssl_protocols configuration option  (Michael Paquier <michael.paquier@gmail.com>)
Responses Re: [PATCH] add ssl_protocols configuration option  (Michael Paquier <michael.paquier@gmail.com>)
List pgsql-hackers
Michael Paquier <michael.paquier@gmail.com> writes:
>>>
>>> Perhaps ssloptions.[ch], unless you plan to add non-option-related code
>>> there later?
>>
>> I don't think anything else than common options-related code fits in
>> there, so ssloptions.c makes sense to me.
>>
>>> BTW, there is no Regent code in your openssl.c, so the copyright
>>> statement is incorrect.
>>
>> Good catch, I just blindly copied that from some other file.
> There have been arguments in favor and against this patch... But
> marking it as returned with feedback because of a lack of activity in
> the last couple of weeks. Feel free to update if you think that's not
> correct, and please move this patch to commit fest 2014-12.

Attached is a new version that addresses the earlier feedback: renamed
the added *.[ch] files and removed incorrect copyright line.

I'm moving this to the current CF.

--
Alex


Attachment

pgsql-hackers by date:

Previous
From: Heikki Linnakangas
Date:
Subject: Re: Commit fest 2014-12, let's begin!
Next
From: Heikki Linnakangas
Date:
Subject: Re: Custom timestamp format in logs