From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Unfriendly handling of pg_hba SSL options with SSL off |
Date: | 2011-04-25 16:59:49 |
Message-ID: | BANLkTikcG36XTkHc3tY5GECt65rvxzkjHA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, Apr 25, 2011 at 12:52 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> A recent complaint in pgsql-novice revealed that if you have say
>
> hostssl all all 127.0.0.1/32 md5 clientcert=1
>
> in pg_hba.conf, but you forget to enable SSL in postgresql.conf,
> you get something like this:
>
> LOG: client certificates can only be checked if a root certificate store is available
> HINT: Make sure the root.crt file is present and readable.
> CONTEXT: line 82 of configuration file "/home/tgl/version90/data/pg_hba.conf"
> LOG: client certificates can only be checked if a root certificate store is available
> HINT: Make sure the root.crt file is present and readable.
> CONTEXT: line 84 of configuration file "/home/tgl/version90/data/pg_hba.conf"
> FATAL: could not load pg_hba.conf
>
> Needless to say, this is pretty unhelpful, especially if you actually do
> have a root.crt file.
>
> I'm inclined to think that the correct fix is to make parse_hba_line,
> where it first realizes the line is "hostssl", check not only that SSL
> support is compiled but that it's turned on. Is it really sensible to
> allow hostssl lines in pg_hba.conf when SSL is turned off? At best
> they are no-ops, and at worst they're going to result in weird failures
> like this one.
It's not clear to me what behavior you are proposing. Would we
disregard the hostssl line or treat it as an error?
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2011-04-25 17:04:52 | Re: branching for 9.2devel |
Previous Message | Robert Haas | 2011-04-25 16:55:21 | Re: Extension Packaging |