Re: Trust intermediate CA for client certificates

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

In response to

Responses

Browse pgsql-general by date

  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

Browse pgsql-hackers by date

  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