Re: .pgpass's behavior has changed

From: Noah Misch <noah(at)leadboat(dot)com>
To: robertmhaas(at)gmail(dot)com
Cc: Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: .pgpass's behavior has changed
Date: 2017-04-30 08:09:28
Message-ID: 20170430080928.GE278614@rfd.leadboat.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, Apr 28, 2017 at 04:54:32PM +0900, Kyotaro HORIGUCHI wrote:
> I noticed that the precedence between host and hostaddr in a
> connection string is reversed in regard to .pgpass lookup in
> devel.
>
> For example the following connection string uses a .pgpass entry
> with "127.0.0.1", not "hoge".
>
> "host=hoge hostaddr=127.0.0.1 port=5432 dbname=postgres"
>
>
> This change was introdueced by the commit
> 274bb2b3857cc987cfa21d14775cae9b0dababa5 and the current behavior
> contradicts the documentation.
>
> https://www.postgresql.org/docs/devel/static/libpq-connect.html
>
> > hostaddr
> > ...
> > Note that authentication is likely to fail if host is not the
> > name of the server at network address hostaddr. Also, note that
> > host rather than hostaddr is used to identify the connection in
> > a password file (see Section 33.15, “The Password File”).
>
> I think this should be fixed for the same reason with the
> following commit.
>
> > commit 11003eb55658df0caf183eef69c7a97d56a4f2d7
> > Author: Robert Haas <rhaas(at)postgresql(dot)org>
> > Date: Thu Dec 1 14:36:39 2016 -0500
> >
> > libpq: Fix inadvertent change in PQhost() behavior.
>
> But the above also leaves a bug so I sent another patch to fix
> it. The attched patch restores the 9.6's beavior of looking up
> .pgpass file in the same manner to the aother patch.

[Action required within three days. This is a generic notification.]

The above-described topic is currently a PostgreSQL 10 open item. Robert,
since you committed the patch believed to have created it, you own this open
item. If some other commit is more relevant or if this does not belong as a
v10 open item, please let us know. Otherwise, please observe the policy on
open item ownership[1] and send a status update within three calendar days of
this message. Include a date for your subsequent status update. Testers may
discover new open items at any time, and I want to plan to get them all fixed
well in advance of shipping v10. Consequently, I will appreciate your efforts
toward speedy resolution. Thanks.

[1] https://www.postgresql.org/message-id/20170404140717.GA2675809%40tornado.leadboat.com

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Ilya Shkuratov 2017-04-30 08:34:48 Re: CTE inlining
Previous Message Noah Misch 2017-04-30 08:05:17 Re: Concurrent ALTER SEQUENCE RESTART Regression