Re: Cutting support for OpenSSL 1.0.1 and 1.0.2 in 17~?

From: Daniel Gustafsson <daniel(at)yesql(dot)se>
To: Michael Paquier <michael(at)paquier(dot)xyz>
Cc: Peter Eisentraut <peter(at)eisentraut(dot)org>, Jacob Champion <jacob(dot)champion(at)enterprisedb(dot)com>, Thomas Munro <thomas(dot)munro(at)gmail(dot)com>, Postgres hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, mikael(dot)kjellstrom(at)gmail(dot)com, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Subject: Re: Cutting support for OpenSSL 1.0.1 and 1.0.2 in 17~?
Date: 2024-05-07 10:36:24
Message-ID: 50C99BEC-BF4F-4470-919F-8925149695E4@yesql.se
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

> On 7 May 2024, at 01:31, Michael Paquier <michael(at)paquier(dot)xyz> wrote:
>
> On Fri, May 03, 2024 at 10:39:15AM +0200, Daniel Gustafsson wrote:
>> They are no-ops when linking against v18, but writing an extension which
>> targets all supported versions of postgres along with their respective
>> supported OpenSSL versions make them still required, or am I missing something?
>
> Yeah, that depends on how much version you expect your application to
> work on. Still it seems to me that there's value in mentioning that
> if your application does not care about anything older than OpenSSL
> 1.1.0, like PG 18 assuming that this patch is merged, then these calls
> are pointless for HEAD. The routine definitions would be around only
> for the .so compatibility.

Fair enough. I've taken a stab at documenting that the functions are
deprecated, while at the same time documenting when and how they can be used
(and be useful). The attached also removes one additional comment in the
testcode which is now obsolete (since removing 1.0.1 support really), and fixes
the spurious whitespace you detected upthread.

--
Daniel Gustafsson

Attachment Content-Type Size
v13-0002-Remove-pg_strong_random-initialization.patch application/octet-stream 1.9 KB
v13-0001-Remove-support-for-OpenSSL-1.0.2.patch application/octet-stream 35.6 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Alexander Korotkov 2024-05-07 11:42:16 Re: Revert: Remove useless self-joins *and* -DREALLOCATE_BITMAPSETS make server crash, regress test fail.
Previous Message Richard Guo 2024-05-07 10:18:54 Re: Revert: Remove useless self-joins *and* -DREALLOCATE_BITMAPSETS make server crash, regress test fail.