From: | Jelte Fennema <postgres(at)jeltef(dot)nl> |
---|---|
To: | Jacob Champion <jchampion(at)timescale(dot)com> |
Cc: | Michael Paquier <michael(at)paquier(dot)xyz>, thomas(at)habets(dot)se, pgsql-hackers(at)postgresql(dot)org, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Bruce Momjian <bruce(at)momjian(dot)us>, Andrew Dunstan <andrew(at)dunslane(dot)net> |
Subject: | Re: [PATCH] Add `verify-system` sslmode to use system CA pool for server cert |
Date: | 2023-01-11 14:37:17 |
Message-ID: | CAGECzQQuO9As_=44_=p425nUt9vvjUi2AwDNaWDfN7dTk4TMAA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
LGTM. As far as I can tell this is ready for a committer.
On Wed, 11 Jan 2023 at 00:15, Jacob Champion <jchampion(at)timescale(dot)com> wrote:
>
> On Mon, Jan 9, 2023 at 7:07 AM Jelte Fennema <postgres(at)jeltef(dot)nl> wrote:
> > I also took a closer look at the code, and the only comment I have is:
> >
> > > appendPQExpBuffer(&conn->errorMessage,
> >
> > These calls can all be replaced by the recently added libpq_append_conn_error
>
> Argh, thanks for the catch. Fixed.
>
> > Finally I tested this against a Postgres server I created on Azure and
> > the new value works as expected. The only thing that I think would be
> > good to change is the error message when sslmode=verify-full, and
> > sslrootcert is not provided, but ~/.postgresql/root.crt is also not available.
> > I think it would be good for the error to mention sslrootcert=system
>
> Good idea. The wording I chose in v6 is
>
> Either provide the file, use the system's trusted roots with
> sslrootcert=system, or change sslmode to disable server certificate
> verification.
>
> What do you think?
>
> Thanks!
> --Jacob
From | Date | Subject | |
---|---|---|---|
Next Message | 斯波隼斗 | 2023-01-11 14:44:11 | Re: can while loop in ClockSweepTick function be kind of infinite loop in some cases? |
Previous Message | Peter Eisentraut | 2023-01-11 14:22:43 | Re: doc: add missing "id" attributes to extension packaging page |