Re: [HACKERS] expand_dbname in postgres_fdw - Mailing list pgsql-hackers

From Robert Haas
Subject Re: [HACKERS] expand_dbname in postgres_fdw
Date
Msg-id CA+TgmobV0LpMaGHdip7DM+sUWVg99G_OfO09t4+pipEGVjS9_Q@mail.gmail.com
Whole thread Raw
In response to Re: [HACKERS] expand_dbname in postgres_fdw  (Ashutosh Bapat <ashutosh.bapat@enterprisedb.com>)
Responses Re: [HACKERS] expand_dbname in postgres_fdw
Re: [HACKERS] expand_dbname in postgres_fdw
List pgsql-hackers
On Wed, Jul 26, 2017 at 5:38 AM, Ashutosh Bapat
<ashutosh.bapat@enterprisedb.com> wrote:
> According to F.34.1.1 at [1] passing connection string as dbname
> option should work, so your question is valid. I am not aware of any
> discussion around this on hackers. Comments in connect_pg_server()
> don't help either. But I guess, we expect users to set up individual
> foreign server and user mapping options instead of putting those in a
> connection string. I can not think of any reason except that it
> improves readability. If postgres_fdw wants to take certain actions
> based on the values of individual options, having them separate is
> easier to handle than parsing them out of a connection string.
>
> Any way, if we are not going to change current behaviour, we should
> change the documentation and say that option dbname means "database
> name" and not a connection string.

I kind of wonder if this had some security aspect to it?  But not sure.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company



pgsql-hackers by date:

Previous
From: Sokolov Yura
Date:
Subject: Re: [HACKERS] Increase Vacuum ring buffer.
Next
From: Tom Lane
Date:
Subject: Re: [HACKERS] Change in "policy" on dump ordering?