On Wed, Mar 10, 2010 at 11:52 AM, Bruce Momjian <br...@momjian.us> wrote:
> The attached patch reports the fact that .pgpass was used if the libpq
> connection fails:

+       /*
+        *      If the connection failed, we should mention that
+        *      we got the password from .pgpass in case that
+        *      password is wrong.
+        */
+       if (conn->used_dot_pgpass && conn->password_needed)
+               appendPQExpBufferStr(&conn->errorMessage,
+                       libpq_gettext("(password retrieved from .pgpass)\n"));

I think that this should be in PQconnectPoll() rather than connectDBComplete().
Otherwise, only when we make a connection to the server in a nonblocking manner
(i.e., use PQconnectStart() and PQconnectPoll()), that HINT message is
not output.
This looks odd for me. Thought?

Regards,

-- 
Fujii Masao
NIPPON TELEGRAPH AND TELEPHONE CORPORATION
NTT Open Source Software Center

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to