Re: Replace current implementations in crypt() and gen_salt() to OpenSSL - Mailing list pgsql-hackers

From Joe Conway
Subject Re: Replace current implementations in crypt() and gen_salt() to OpenSSL
Date
Msg-id 677c6e1e-217d-419e-a6eb-9b87d0fcbcf6@joeconway.com
Whole thread Raw
In response to Re: Replace current implementations in crypt() and gen_salt() to OpenSSL  (Daniel Gustafsson <daniel@yesql.se>)
Responses Re: Replace current implementations in crypt() and gen_salt() to OpenSSL
List pgsql-hackers
On 10/29/24 10:08, Daniel Gustafsson wrote:
>> On 29 Oct 2024, at 13:53, Joe Conway <mail@joeconway.com> wrote:
>> On 10/29/24 05:57, Daniel Gustafsson wrote:
>>>> On 26 Oct 2024, at 20:10, Joe Conway <mail@joeconway.com> wrote:

>>>> Rather than depend on figuring out if we are in FIPS_mode in a
>>>> portable way, I think the GUC is simpler and sufficient. Why
>>>> not do that and just use a better name, e.g.
>>>> legacy_crypto_enabled or something similar (bike-shedding
>>>> welcomed) as in the attached.

>>> I'm not very enthusiastic about adding a GUC to match a system property like
>>> that for the same reason why we avoid GUCs with transitive dependencies.
>>> Re-reading the thread and thinking about I think the best solution would be to
>>> split these functions off into their own extension.

>> Seems like that would be an issue for backward comparability and upgrades.

> That's undoubtedly a downside of this proposal which the GUC proposal doesn't have.

Any other opinions out there?

-- 
Joe Conway
PostgreSQL Contributors Team
RDS Open Source Databases
Amazon Web Services: https://aws.amazon.com



pgsql-hackers by date:

Previous
From: Fujii Masao
Date:
Subject: Re: Improve error messages for database object stats manipulation functions during recovery
Next
From: Michel Pelletier
Date:
Subject: Re: Using Expanded Objects other than Arrays from plpgsql