Re: pg_auth_members.grantor is bunk - Mailing list pgsql-hackers

From Robert Haas
Subject Re: pg_auth_members.grantor is bunk
Date
Msg-id CA+TgmobLM8mBPBv20KidZ0tMfS2J=kVQ-OE7L7ynd5qoTJRW_Q@mail.gmail.com
Whole thread Raw
In response to Re: pg_auth_members.grantor is bunk  ("David G. Johnston" <david.g.johnston@gmail.com>)
Responses Re: pg_auth_members.grantor is bunk  (Robert Haas <robertmhaas@gmail.com>)
Re: pg_auth_members.grantor is bunk  (Robert Haas <robertmhaas@gmail.com>)
List pgsql-hackers
On Fri, Jun 24, 2022 at 4:30 PM David G. Johnston
<david.g.johnston@gmail.com> wrote:
>> Upthread, I proposed that "drop role baz" should fail here
>
> I concur with this.
>
> I think that the grantor owns the grant, and that REASSIGNED OWNED should be able to move those grants to someone
else.
>
> By extension, DROP OWNED should remove them.

Interesting. I hadn't thought about changing the behavior of DROP
OWNED BY and REASSIGN OWNED BY. A quick experiment supports your
interpretation:

rhaas=# grant select on table foo to bar;
GRANT
rhaas=# revoke select on table foo from bar;
REVOKE
rhaas=# grant select on table foo to bar with grant option;
GRANT
rhaas=# set role bar;
SET
rhaas=> grant select on table foo to baz;
GRANT
rhaas=> reset role;
RESET
rhaas=# drop role bar;
ERROR:  role "bar" cannot be dropped because some objects depend on it
DETAIL:  privileges for table foo
rhaas=# drop owned by bar;
DROP OWNED
rhaas=# drop role bar;
DROP ROLE

So, privileges on tables (and presumably all other SQL objects)
already work the way that you propose here. If we choose to make role
memberships work in some other way then the two will be inconsistent.
Probably we shouldn't do that. There is still the question of what the
SQL specification says about this, but I would guess that it mandates
the same behavior for all kinds of privileges rather than treating
role memberships and table permissions in different ways. I could be
wrong, though.

-- 
Robert Haas
EDB: http://www.enterprisedb.com



pgsql-hackers by date:

Previous
From: "David G. Johnston"
Date:
Subject: Re: pg_auth_members.grantor is bunk
Next
From: Zhihong Yu
Date:
Subject: Re: [PATCH] Optimize json_lex_string by batching character copying