> 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[].
