> On Mon, Feb 23, 2026 at 03:32:02PM +0100, Daniel Gustafsson wrote: > +PG_FUNCTION_INFO_V1(ssl_supported_groups); > +Datum > +ssl_supported_groups(PG_FUNCTION_ARGS) > The supported groups is sent as a TLS extension, and we already have > ssl_extension_info() for listing extension values. Shouldn't this be a part > of > that function for consistency?
Good point, thanks. I'll move the functionality in there. > --- /dev/null > +++ b/contrib/sslinfo/sslinfo--1.3.sql > @@ -0,0 +1,56 @@ > +/* contrib/sslinfo/sslinfo--1.3.sql */ > No need to spin a new 1.3 file, the upgrade file from 1.2 to 1.3 is sufficient > since the extension machinery will play the files in order to reach 1.3. > > > +CREATE FUNCTION ssl_shared_groups() RETURNS text > Why not return this as text[] rather than a delimited string? No particular reason, but I see the point -- going to change it to text[].
