Re: PSQL - prevent describe listing tables that are already in listed schemas

From: vignesh C <vignesh21(at)gmail(dot)com>
To: Peter Smith <smithpb2250(at)gmail(dot)com>
Cc: Nisha Moond <nisha(dot)moond412(at)gmail(dot)com>, Jim Jones <jim(dot)jones(at)uni-muenster(dot)de>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: PSQL - prevent describe listing tables that are already in listed schemas
Date: 2026-07-08 11:06:20
Message-ID: CALDaNm3Cf9vYAFa3XYHMjJSaaXaZygKxgc7bahMrC2JBvA7pKg@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, 8 Jul 2026 at 04:58, Peter Smith <smithpb2250(at)gmail(dot)com> wrote:
>
> A rebase was needed.
>
> PSA v7.

One minor comment:
Can we add a comment here too, similar to how you have added for
describe publications:
@@ -3048,7 +3048,12 @@ describeOneTableDetails(const char *schemaname,
"FROM
pg_catalog.pg_publication p\n"
" JOIN
pg_catalog.pg_publication_rel pr ON p.oid = pr.prpubid\n"
" JOIN
pg_catalog.pg_class c ON c.oid = pr.prrelid\n"
- "WHERE
pr.prrelid = '%s'\n",
+ "WHERE
pr.prrelid = '%s'\n"
+ " AND NOT EXISTS (\n"
+ " SELECT 1\n"
+ " FROM
pg_catalog.pg_publication_namespace pn\n"
+ " WHERE
pn.pnpubid = p.oid\n"
+ " AND
pn.pnnspid = c.relnamespace)\n",
oid, oid, oid);

Regards,
Vignesh

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Tatsuro Yamada 2026-07-08 11:09:51 RE: Add enable_groupagg GUC parameter to control GroupAggregate usage
Previous Message Matthias van de Meent 2026-07-08 10:57:00 Re: Increased SECURITY RISCS from omitting some compikler options when building PG with meson; e.g. -fcf-protection=full