From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com> |
Cc: | sfrost(at)snowman(dot)net, mikael(dot)gustavsson(at)smhi(dot)se, magnus(at)hagander(dot)net, pgsql-general(at)postgresql(dot)org, peter(dot)svensson(at)smhi(dot)se |
Subject: | Re: Problem with ssl and psql in Postgresql 13 |
Date: | 2020-12-24 16:54:32 |
Message-ID: | 795101.1608828872@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-general |
I wrote:
> Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com> writes:
>> The attached the first patch does that.
> +1, it seems like a clear oversight that the GSSENC patches didn't adjust
> these messages. The reason SSL state is mentioned is that it's relevant
> to which pg_hba entry gets chosen; and once we invented "hostgssenc"
> entries, GSSENC state is also relevant.
Thinking a little more about that: there are not four distinct states,
because GSS and SSL can't both be enabled (ProcessStartupPacket enforces
that). So I propose that instead of adding a new field, we make the
existing field say one of three things: "GSS encryption", "SSL
encryption", or "no encryption". As per attached. In the back branches,
it might be best to spell these as "GSS encryption", "SSL on", and "SSL
off", just to minimize the cosmetic change.
regards, tom lane
Attachment | Content-Type | Size |
---|---|---|
improve-pg_hba-rejection-messages-2.patch | text/x-diff | 4.1 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Open _ | 2020-12-24 17:20:29 | created type not found in create table |
Previous Message | Adrian Klaver | 2020-12-24 16:52:47 | Re: Missing rows after migrating from postgres 11 to 12 with logical replication |