Re: [PATCH] add ssl_protocols configuration option
Alex Shulgin <ash@commandprompt.com>
From: Alex Shulgin <ash@commandprompt.com>
To: Michael Paquier <michael.paquier@gmail.com>
Cc: Dag-Erling Smørgrav <des@des.no>, PostgreSQL mailing lists <pgsql-hackers@postgresql.org>
Date: 2014-12-15T14:15:56Z
Lists: pgsql-hackers
Attachments
- ssl-protocols-v3.diff (text/x-diff) patch v3
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