> On 14 Jan 2025, at 13:12, Hayato Kuroda (Fujitsu) <[email protected]> > wrote:
> Similar with [1], `pg_gen_salt_rounds` is not an SQL function. > I think we do not have to mention the function because it's just another > implementation of gen_salt(). > Also, use <function> instead of <literal>. Fixed. > I think we must call MarkGUCPrefixReserved() to catch the mis-spell. Good point, fixed. > Also: I'm not sure whether we should bump the version of pgcrypto. It should > be done when > the API is changed, but the patch does not do. Thought? I don't think this constitutes a change which warrants a version bump so I've left that out for now. The attached includes a rename from "legacy_crypto" to "builtin_crypto". While legacy might apply now, there is work ongoing to modernize the algorithms supported by crypt [0] so legacy might not be applicable soon (this GUC would however still be relevant as the proposed code isn't FIPS certified). Builtin seems like a more future-proof choice in terms of naming. -- Daniel Gustafsson [0] [email protected]
v8-0001-pgcrypto-Make-it-possible-to-disable-built-in-cry.patch
Description: Binary data
