Re: [PATCH] HINT: pg_hba.conf changed since last config reload - Mailing list pgsql-hackers

From Peter Eisentraut
Subject Re: [PATCH] HINT: pg_hba.conf changed since last config reload
Date
Msg-id 545BFA52.8080903@gmx.net
Whole thread Raw
In response to [PATCH] HINT: pg_hba.conf changed since last config reload  (Craig Ringer <craig@2ndquadrant.com>)
Responses Re: [PATCH] HINT: pg_hba.conf changed since last config reload  (Robert Haas <robertmhaas@gmail.com>)
Re: [PATCH] HINT: pg_hba.conf changed since last config reload  (Bruce Momjian <bruce@momjian.us>)
Re: [PATCH] HINT: pg_hba.conf changed since last config reload  (Alex Shulgin <ash@commandprompt.com>)
List pgsql-hackers
On 10/16/14 11:34 PM, Craig Ringer wrote:
> psql: FATAL:  Peer authentication failed for user "fred"
> HINT:  See the server error log for additional information.

I think this is wrong for many reasons.

I have never seen an authentication system that responds with, hey, what
you just did didn't get you in, but the administrators are currently in
the process of making a configuration change, so why don't you check
that out.

We don't know whether the user has access to the server log.  They
probably don't.  Also, it is vastly more likely that the user really
doesn't have access in the way they chose, so throwing in irrelevant
hints will be distracting.

Moreover, it will be confusing to regular users if this message
sometimes shows up and sometimes doesn't, independent of their own state
and actions.

Finally, the fact that a configuration change is in progress is
privileged information.  Unprivileged users can deduct from the presence
of this message that administrators are doing something, and possibly
that they have done something wrong.

I think it's fine to log a message in the server log if the pg_hba.conf
file needs reloading.  But the client shouldn't know about this at all.




pgsql-hackers by date:

Previous
From: Andrew Dunstan
Date:
Subject: Re: json, jsonb, and casts
Next
From: Michael Paquier
Date:
Subject: Re: split builtins.h to quote.h