From: | Jacob Champion <jchampion(at)timescale(dot)com> |
---|---|
To: | thomas(at)habets(dot)se |
Cc: | 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: | 2022-11-08 01:04:14 |
Message-ID: | CAAWbhmiHRade=v-L4srAgi7bYLS5mErNMz2OUBjwmxGxaBPMDA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Thu, Nov 3, 2022 at 4:39 PM Jacob Champion <jchampion(at)timescale(dot)com> wrote:
> There is an additional test failure with LibreSSL, which doesn't appear
> to honor the SSL_CERT_FILE environment variable. This isn't a problem in
> production -- if you're using LibreSSL, you'd presumably understand that
> you can't use that envvar -- but it makes testing difficult, because I
> don't yet know a way to tell LibreSSL to use a different set of roots
> for the duration of a test. Has anyone dealt with this before?
Fixed in v3, with a large hammer (configure-time checks). Hopefully
I've missed a simpler solution.
> > If there are no valuable use cases for weaker checks, then we could go
> > even further than my 0002 and just reject any weaker sslmodes
> > outright. That'd be nice.
Done. sslrootcert=system now prevents you from explicitly setting a
weaker sslmode, to try to cement it as a Do What I Mean sort of
feature. If you need something weird then you can still jump through
the hoops by setting sslrootcert to a real file, same as today.
The macOS/OpenSSL 3.0.0 failure is still unfixed.
Thanks,
--Jacob
Attachment | Content-Type | Size |
---|---|---|
since-v2.diff.txt | text/plain | 16.7 KB |
v3-0001-libpq-add-sslrootcert-system-to-use-default-CAs.patch | text/x-patch | 21.2 KB |
v3-0002-libpq-force-sslmode-verify-full-for-system-CAs.patch | text/x-patch | 8.8 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2022-11-08 01:04:16 | Re: Allow file inclusion in pg_hba and pg_ident files |
Previous Message | Peter Geoghegan | 2022-11-08 00:34:27 | Re: Allow single table VACUUM in transaction block |