Re: Add pg_file_sync() to adminpack

Julien Rouhaud <[email protected]> Tue, 14 Jan 2020 23:12:18 +0100
Newsgroups gmane.comp.db.postgresql.devel.general
Message-ID <CAOBaU_bscJbeHMuMgu3z2HdT1f49z6cmXtbaBiLH+QyEwi-i+g@mail.gmail.com>
--0000000000005fad67059c20e330
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Le mar. 14 janv. 2020 =C3=A0 22:57, Stephen Frost <[email protected]> a =
=C3=A9crit :

> Greetings,
>
> * Julien Rouhaud ([email protected]) wrote:
> > On Fri, Jan 10, 2020 at 10:50 AM Fujii Masao <[email protected]>
> wrote:
> > > On Thu, Jan 9, 2020 at 10:39 PM Julien Rouhaud <[email protected]>
> wrote:
> > > > I think that pg_write_server_files should be allowed to call that
> > > > function by default.
> > >
> > > But pg_write_server_files users are not allowed to execute
> > > other functions like pg_file_write() by default. So doing that
> > > change only for pg_file_sync() looks strange to me.
> >
> > Ah indeed.  I'm wondering if that's an oversight of the original
> > default role patch or voluntary.
>
> It was intentional.
>

ok, thanks for the clarification.

>

--0000000000005fad67059c20e330
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"auto"><div><div class=3D"gmail_quote"><div dir=3D"ltr" class=3D=
"gmail_attr">Le mar. 14 janv. 2020 =C3=A0 22:57, Stephen Frost &lt;<a href=
=3D"mailto:[email protected]">[email protected]</a>&gt; a =C3=A9crit=C2=
=A0:<br></div><blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;=
border-left:1px #ccc solid;padding-left:1ex">Greetings,<br>
<br>
* Julien Rouhaud (<a href=3D"mailto:[email protected]" target=3D"_blank" r=
el=3D"noreferrer">[email protected]</a>) wrote:<br>
&gt; On Fri, Jan 10, 2020 at 10:50 AM Fujii Masao &lt;<a href=3D"mailto:mas=
[email protected]" target=3D"_blank" rel=3D"noreferrer">masao.fujii@gmail.=
com</a>&gt; wrote:<br>
&gt; &gt; On Thu, Jan 9, 2020 at 10:39 PM Julien Rouhaud &lt;<a href=3D"mai=
lto:[email protected]" target=3D"_blank" rel=3D"noreferrer">rjuju123@gmail=
.com</a>&gt; wrote:<br>
&gt; &gt; &gt; I think that pg_write_server_files should be allowed to call=
 that<br>
&gt; &gt; &gt; function by default.<br>
&gt; &gt;<br>
&gt; &gt; But pg_write_server_files users are not allowed to execute<br>
&gt; &gt; other functions like pg_file_write() by default. So doing that<br=
>
&gt; &gt; change only for pg_file_sync() looks strange to me.<br>
&gt; <br>
&gt; Ah indeed.=C2=A0 I&#39;m wondering if that&#39;s an oversight of the o=
riginal<br>
&gt; default role patch or voluntary.<br>
<br>
It was intentional.<br></blockquote></div></div><div dir=3D"auto"><br></div=
><div dir=3D"auto">ok, thanks for the clarification.=C2=A0</div><div dir=3D=
"auto"><div class=3D"gmail_quote"><blockquote class=3D"gmail_quote" style=
=3D"margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
</blockquote></div></div></div>

--0000000000005fad67059c20e330--