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-04-27 18:32:19
Message-ID: 56D38F23-11BB-4A30-866F-9D6EA21ABA25@yesql.se
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

> On 25 Apr 2024, at 05:49, Michael Paquier <michael(at)paquier(dot)xyz> wrote:
>
> On Wed, Apr 24, 2024 at 01:31:12PM +0200, Daniel Gustafsson wrote:
>> Done. Attached are the two remaining patches, rebased over HEAD, for removing
>> support for OpenSSL 1.0.2 in v18. Parking them in the commitfest for now.
>
> You have mentioned once upthread the documentation of PQinitOpenSSL():
> However, this is unnecessary when using <productname>OpenSSL</productname>
> version 1.1.0 or later, as duplicate initializations are no longer problematic.
>
> With 1.0.2's removal in place, this could be simplified more and the
> patch does not touch it. This relates also to pq_init_crypto_lib,
> which is gone with 0001. Of course, it is not possible to just remove
> PQinitOpenSSL() or old application may fail linking. Removing
> pq_init_crypto_lib reduces any confusion around this part of the
> initialization.

That's a good point, there is potential for more code removal here. The
attached 0001 takes a stab at it while it's fresh in mind, I'll revisit before
the July CF to see if there is more that can be done.

--
Daniel Gustafsson

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Daniel Gustafsson 2024-04-27 18:33:55 Re: Cutting support for OpenSSL 1.0.1 and 1.0.2 in 17~?
Previous Message David G. Johnston 2024-04-27 17:46:16 Re: Read table rows in chunks