Skip site navigation (1) Skip section navigation (2)

Re: Warning about invalid .pgpass passwords

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Bruce Momjian <bruce(at)momjian(dot)us>
Cc: Alvaro Herrera <alvherre(at)commandprompt(dot)com>, Eamonn Martin <mas01em(at)gold(dot)ac(dot)uk>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Warning about invalid .pgpass passwords
Date: 2010-03-10 03:39:27
Message-ID: (view raw, whole thread or download thread mbox)
Lists: pgsql-hackers
Bruce Momjian <bruce(at)momjian(dot)us> writes:
> The attached patch reports the fact that .pgpass was used if the libpq
> connection fails:

The test is in a very inappropriate place --- it will be missed by
several fully-supported code paths.

> I am not sure if I like the parentheses or not.

I don't like 'em.  If you don't have enough confidence in the message to
be replacing the normal error string, you probably shouldn't be doing
this at all.  We'll look silly if we attach such a comment to a message
that indicates a network failure, for example; and cases where the
comment is actively misleading would be even worse.

I'm inclined to think that maybe we should make the server return a
distinct SQLSTATE for "bad password", and have libpq check for that
rather than just assuming that the failure must be bad password.
The main argument against this is probably that it would tell a bad
guy that he's got a valid username but not a valid password.  Not
sure if that's a serious issue or not --- I seem to recall that we
are exposing validity of user names already (or was that database
names?).  It would also mean that the new message only triggers when
talking to a 9.0+ server, but since we've gotten along without it
till now, that aspect doesn't bother me at all.

A compromise would be to check for
with the knowledge that we got asked for a password would give
fairly high confidence though not certainty that the problem is a bad

			regards, tom lane

In response to


pgsql-hackers by date

Next:From: Josh BerkusDate: 2010-03-10 06:02:09
Subject: PD_ALL_VISIBLE flag error on 9.0 alpha 4
Previous:From: Fujii MasaoDate: 2010-03-10 03:31:56
Subject: Re: Warning about invalid .pgpass passwords

Privacy Policy | About PostgreSQL
Copyright © 1996-2017 The PostgreSQL Global Development Group