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 34E10ADF-C350-481E-91EB-7548F1EDA722@yesql.se
обсуждение исходный текст
Ответ на pgsql: Only provide new libpq sslpasskey hook for openssl-enabled build  (Andrew Dunstan <andrew@dunslane.net>)
Ответы Re: pgsql: Only provide new libpq sslpasskey hook for openssl-enabledbuild  (Andrew Dunstan <andrew.dunstan@2ndquadrant.com>)
Список pgsql-committers
> 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.

cheers ./daniel

[0] E1BB5261-5C89-494A-841B-283DAD27A2EA@yesql.se


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

Предыдущее
От: Andrew Dunstan
Дата:
Сообщение: pgsql: Use a slightly more liberal regex to detect Visual Studio versio
Следующее
От: Andrew Dunstan
Дата:
Сообщение: Re: pgsql: Only provide new libpq sslpasskey hook for openssl-enabledbuild