Re: [HACKERS] expand_dbname in postgres_fdw

Поиск
Список
Период
Сортировка
От Robert Haas
Тема Re: [HACKERS] expand_dbname in postgres_fdw
Дата
Msg-id CA+TgmobV0LpMaGHdip7DM+sUWVg99G_OfO09t4+pipEGVjS9_Q@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [HACKERS] expand_dbname in postgres_fdw  (Ashutosh Bapat <ashutosh.bapat@enterprisedb.com>)
Ответы Re: [HACKERS] expand_dbname in postgres_fdw  (Alvaro Herrera <alvherre@2ndquadrant.com>)
Re: [HACKERS] expand_dbname in postgres_fdw  (Tom Lane <tgl@sss.pgh.pa.us>)
Список 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 по дате отправления:

Предыдущее
От: Sokolov Yura
Дата:
Сообщение: Re: [HACKERS] Increase Vacuum ring buffer.
Следующее
От: Tom Lane
Дата:
Сообщение: Re: [HACKERS] Change in "policy" on dump ordering?