On Thu, Nov 27, 2014 at 8:51 PM, Alex Shulgin <[email protected]> wrote: > > Dag-Erling Smørgrav <[email protected]> writes: > >> Alex Shulgin <[email protected]> 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 ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
