Re: BUG #17377: only superusers can query or manipulate replication origins

From: Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>
To: chaolpan(at)cisco(dot)com, pgsql-bugs(at)lists(dot)postgresql(dot)org
Subject: Re: BUG #17377: only superusers can query or manipulate replication origins
Date: 2022-02-04 04:54:40
Message-ID: CAD21AoB62=BfFT-0uBpYk=2TaG9mF5gvJdLu0xUVTKr5_u0+FQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

On Sat, Jan 22, 2022 at 10:09 PM PG Bug reporting form
<noreply(at)postgresql(dot)org> wrote:
>
> The following bug has been logged on the website:
>
> Bug reference: 17377
> Logged by: James Pang
> Email address: chaolpan(at)cisco(dot)com
> PostgreSQL version: 13.4
> Operating system: RHEL8.4
> Description:
>
> PG 13.4 on RHEL8.4 ,
> we have a user with REPLICATION for logical replication , try to handle
> "replication change loop back" by set replication origin,
>
> select pg_replication_origin_create('test');
> ERROR: only superusers can query or manipulate replication origins
>
> Is it possible to "remove superuser() check" for pg_replication related
> functions and view query, we expect any role with replication can
> create/drop origin, and roles with pg_monitor can query the progress.
>
> we tested that PG 14.1 works if we grant execution privileges ,

Yeah, we removed superuser checks from replication origin functions in PG14.

> is it
> possible to merge the similar enhancement to Postgresql 13.5 or next minor
> version ? so we can use that in pg13 version.

Hmm, I don't think this change will be back-patched since it's not a bug fix.

Regards,

--
Masahiko Sawada
EDB: https://www.enterprisedb.com/

In response to

Browse pgsql-bugs by date

  From Date Subject
Next Message Masahiko Sawada 2022-02-04 08:48:41 Re: BUG #17385: "RESET transaction_isolation" inside serializable transaction causes Assert at the transaction end
Previous Message Andres Freund 2022-02-04 04:19:35 Re: BUG #17255: Server crashes in index_delete_sort_cmp() due to race condition with vacuum