Re: pgsql: Only provide new libpq sslpasskey hook for openssl-enabledbuild

Поиск
Список
Период
Сортировка
От Daniel Gustafsson
Тема Re: pgsql: Only provide new libpq sslpasskey hook for openssl-enabledbuild
Дата
Msg-id 0E1FF8E3-5A16-481C-9261-9347039DCEB3@yesql.se
обсуждение исходный текст
Ответ на Re: pgsql: Only provide new libpq sslpasskey hook for openssl-enabledbuild  (Andrew Dunstan <andrew.dunstan@2ndquadrant.com>)
Список pgsql-committers
> On 17 Apr 2020, at 21:46, Andrew Dunstan <andrew.dunstan@2ndquadrant.com> wrote:
>
> On 4/17/20 3:27 PM, Daniel Gustafsson wrote:
>>> On 17 Apr 2020, at 20:19, Andrew Dunstan <andrew@dunslane.net> wrote:
>>>
>>> Only provide new libpq sslpasskey hook for openssl-enabled builds
>>>
>>> In commit 4dc6355210 I neglected to put #ifdef USE_OPENSSL around the
>>> declarations of the new items. This is remedied here.
>> This wasn't really the problem I reported though.  As per my patch [0] in the
>> email, the issue is with openssl_tls_init_hook in src/include/libpq/libpq-be.h
>> requiring an OpenSSL specific SSL_CTX parameter, thus breaking compilation for
>> other TLS backends. The name also makes it somewhat backend specific =)
>>
>> PQgetSSLKeyPassHook is written generic enough that it could make sense for
>> other backends too, but for now I think we can keep the guards introduced in
>> this commit.
>
> Oh, *sigh* brainfade.
>
> Will fix.

Thanks, f342d7ad03e61a1cea0339cf0c5aea0d01c3b43e was more what I had in mind.

cheers ./daniel


В списке pgsql-committers по дате отправления:

Предыдущее
От: Andrew Dunstan
Дата:
Сообщение: pgsql: Only provide openssl_tls_init_hook if building with openssl
Следующее
От: Tom Lane
Дата:
Сообщение: Re: pgsql: Only provide openssl_tls_init_hook if building with openssl