Re: Error attribution in foreign scans

From: Itagaki Takahiro <itagaki(dot)takahiro(at)gmail(dot)com>
To: Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com>
Cc: Noah Misch <noah(at)leadboat(dot)com>, hanada(at)metrosystems(dot)co(dot)jp, pgsql-hackers(at)postgresql(dot)org
Subject: Re: Error attribution in foreign scans
Date: 2011-02-09 01:55:05
Message-ID: AANLkTikn6XqjvnW+XsdV3m=Jry81Ye8XXGOKnZhAto52@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, Feb 7, 2011 at 22:47, Heikki Linnakangas
<heikki(dot)linnakangas(at)enterprisedb(dot)com> wrote:
> On Mon, Feb 7, 2011 at 21:17, Noah Misch <noah(at)leadboat(dot)com> wrote:
>> The message does not show which foreign table yielded the error.  We could evade
>> the problem in this case by adding a file name to the error message in the COPY
>> code,

> Yeah, an error context callback like that makes sense. In the case of the
> file FDW, though, just including the filename in the error message seems
> even better. Especially if the error is directly related to failure in
> reading the file.

What do you think about filenames in terms of security? We will allow
non-superusers to use existing foreign tables of file_fdw.
For reference, we hide some path settings in GUC variables.

We also reconsider privilege of fdwoptions, umoptions, etc. They could
contain password or server-side path, but all users can retrieve the
values. It's an existing issue, but will be more serious in 9.1.

--
Itagaki Takahiro

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2011-02-09 01:59:02 Re: Extensions versus pg_upgrade
Previous Message Glenn Maynard 2011-02-09 01:50:06 Re: Issues with generate_series using integer boundaries