From: | Daniel Gustafsson <daniel(at)yesql(dot)se> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Thomas Munro <thomas(dot)munro(at)gmail(dot)com>, Andrew Dunstan <andrew(at)dunslane(dot)net>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, Jacob Champion <jacob(dot)champion(at)enterprisedb(dot)com> |
Subject: | Re: disabled SSL log_like tests |
Date: | 2025-05-08 11:20:06 |
Message-ID: | D9A0285C-D051-4E6F-B9F0-09CE3282EA54@yesql.se |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
> On 7 May 2025, at 23:54, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>
> Daniel Gustafsson <daniel(at)yesql(dot)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
From | Date | Subject | |
---|---|---|---|
Next Message | Aleksander Alekseev | 2025-05-08 12:20:49 | Re: Valgrind - showing memory leaks? |
Previous Message | Richard Guo | 2025-05-08 10:46:11 | Re: PG 18 release notes draft committed |