On Thu, Nov 27, 2014 at 8:51 PM, Alex Shulgin <a...@commandprompt.com> wrote: > > Dag-Erling Smørgrav <d...@des.no> writes: > >> Alex Shulgin <a...@commandprompt.com> writes: >>> OK, looks like I've come up with something workable: I've added >>> sslprotocol connection string keyword similar to pre-existing >>> sslcompression, etc. Please see attached v2 of the original patch. >>> I'm having doubts about the name of openssl.h header though, >>> libpq-openssl.h? >> >> 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. -- Michael
-- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
