From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | Craig Ringer <craig(at)2ndquadrant(dot)com>, Stephen Frost <sfrost(at)snowman(dot)net>, Ian Pilcher <arequipeno(at)gmail(dot)com>, stellr(at)vt(dot)edu, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Trust intermediate CA for client certificates |
Date: | 2013-12-02 17:59:41 |
Message-ID: | 9917.1386007181@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-general pgsql-hackers |
Bruce Momjian <bruce(at)momjian(dot)us> writes:
> I have updated the patch, attached, to be clearer about the requirement
> that intermediate certificates need a chain to root certificates.
I see that you removed the sentence
The root
certificate should be included in every case where
<filename>postgresql.crt</> contains more than one certificate.
in both places where it appeared. I seem to remember that I'd put that
in on the basis of experimentation, ie it didn't work to provide just
a partial chain. You appear to be telling people that it's safe to
omit the root cert, and I think this is wrong.
Specifically, rather than the text "trusted by the server, i.e. signed by
a certificate in the server's <filename>root.crt</filename> file", I think
you need to say "trusted by the server, i.e., appears in the server's
<filename>root.crt</filename> file". Have you experimented with the
configuration you're proposing, and if so, with which OpenSSL versions?
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Zev Benjamin | 2013-12-02 18:15:39 | Re: unnest on multi-dimensional arrays |
Previous Message | Karsten Hilbert | 2013-12-02 17:57:53 | Re: [GENERAL] pg_upgrade ?deficiency |
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2013-12-02 18:19:10 | Re: In-core regression tests for replication, cascading, archiving, PITR, etc. |
Previous Message | Andres Freund | 2013-12-02 17:58:08 | Re: Draft release notes for 9.3.2 |