Re: Extension security improvement: Add support for extensions with an owned schema

From: "Jelte Fennema-Nio" <postgres(at)jeltef(dot)nl>
To: "Robert Haas" <robertmhaas(at)gmail(dot)com>
Cc: "Artem Gavrilov" <artem(dot)gavrilov(at)percona(dot)com>, "Jelte Fennema-Nio" <me(at)jeltef(dot)nl>, "Tomas Vondra" <tomas(at)vondra(dot)me>, "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com>, "Jeff Davis" <pgsql(at)j-davis(dot)com>, "PostgreSQL-development" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Extension security improvement: Add support for extensions with an owned schema
Date: 2025-09-01 14:44:41
Message-ID: DCHJLTNNRZ9A.3257REHC1L4GQ@jeltef.nl
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon Aug 11, 2025 at 9:23 PM CEST, Robert Haas wrote:
> On Mon, Aug 11, 2025 at 1:55 PM Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
>> [ some review ]

Attached is a patch that addresses your comments I think. I restructured
the schema creation code, and added more comments to the
AlterExtensionNamespace code (I couldn't find a way to make the
structure clearer).

> Another thing that's occurring to me here is that nothing prevents
> other objects from making their way into the owned schema. Sure, if we
> create a new schema with nobody having any permissions, then only the
> creating role or some role that has its privileges can add anything in
> there. But that could happen by accident, or privileges could later be
> granted and somebody could add something into the extension schema
> after that. I wonder whether we should lock this down tighter somehow
> and altogether forbid creating objects in that schema except from an
> extension create/upgrade script for the owning extension.

I think that's an interesting idea, and I started with a change to try
this out, that I intend to finish soon. It doesn't seem strictly
necessary, though.

Attachment Content-Type Size
v6-0001-Add-support-for-extensions-with-an-owned-schema.patch text/x-patch 40.9 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2025-09-01 15:05:49 Re: new warnings with clang-21 / how const is Datum
Previous Message Bertrand Drouvot 2025-09-01 14:11:47 Re: Per backend relation statistics tracking