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 87zjbc6eul.fsf@commandprompt.com
Whole thread Raw
In response to Re: [PATCH] add ssl_protocols configuration option  (Dag-Erling Smørgrav <des@des.no>)
Responses Re: [PATCH] add ssl_protocols configuration option  (Michael Paquier <michael.paquier@gmail.com>)
List pgsql-hackers
Dag-Erling Smørgrav <des@des.no> writes:

> Alex Shulgin <ash@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.

--
Alex



pgsql-hackers by date:

Previous
From: Simon Riggs
Date:
Subject: Re: [v9.5] Custom Plan API
Next
From: Mats Erik Andersson
Date:
Subject: initdb: Improve error recovery.