Re: disabled SSL log_like tests - Mailing list pgsql-hackers

From Daniel Gustafsson
Subject Re: disabled SSL log_like tests
Date
Msg-id D9A0285C-D051-4E6F-B9F0-09CE3282EA54@yesql.se
Whole thread Raw
In response to Re: disabled SSL log_like tests  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: disabled SSL log_like tests
List pgsql-hackers
> On 7 May 2025, at 23:54, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>
> Daniel Gustafsson <daniel@yesql.se> writes:
>> I haven't looked at the test in question yet, but we do skip some SSL tests if
>> running against libressl already so I assume this will be able to follow the
>> same pattern.
>
> Ah, thanks for the tip.  I propose the attached, which disables the
> RSA-PSS test altogether on LibreSSL, and modifies the
> intermediate-cert test to accept the result we're actually getting
> on LibreSSL.  We could revert that one if anyone can figure out
> how to make it better, but I don't wish to put any more time into
> it myself.

LGTM for now.

> +# Determine whether this build uses OpenSSL or LibreSSL. As a heuristic, the
> +# HAVE_SSL_CTX_SET_CERT_CB macro isn't defined for LibreSSL.
> +my $libressl = not check_pg_config("#define HAVE_SSL_CTX_SET_CERT_CB 1");

Longer term it would be nice to move this into SSL::Server and have the module
export a function or symbol which returns the underlying library and version,
but that's not for this patch.

> +
> +# As of 5/2025, LibreSSL doesn't actually work for RSA-PSS certificates.

Should we add a link to the relevant thread for future readers?  OpenBSD refer
to MARC for archiving which I believe is stable enough for an inclusion.

    https://marc.info/?l=libressl&m=174664225002441&w=2

--
Daniel Gustafsson




pgsql-hackers by date:

Previous
From: Richard Guo
Date:
Subject: Re: PG 18 release notes draft committed
Next
From: Aleksander Alekseev
Date:
Subject: Re: Valgrind - showing memory leaks?