Horton

Re: Allow CURRENT_ROLE in GRANTED BY

Daniel Gustafsson <daniel@yesql.se>

View thread View on postgr.es
From: Daniel Gustafsson <daniel@yesql.se>
To: Peter Eisentraut <peter.eisentraut@enterprisedb.com>
Cc: Simon Riggs <simon@2ndquadrant.com>, Vik Fearing <vik@postgresfriends.org>, pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2021-11-26T13:17:05Z
Lists: pgsql-hackers 
> On 18 Nov 2021, at 14:42, Daniel Gustafsson <daniel@yesql.se> wrote:
> 
>> On 18 Nov 2021, at 14:41, Peter Eisentraut <peter.eisentraut@enterprisedb.com> wrote:
>> 
>> On 16.11.21 15:27, Daniel Gustafsson wrote:
>>>>> On 16 Nov 2021, at 15:04, Daniel Gustafsson <daniel@yesql.se> wrote:
>>>> ..or should the attached small diff be applied to fix it?
>>> Actually it shouldn't, I realized when hitting Send that it was the wrong
>>> version.  The attached is the proposed diff.
>> 
>> This appears to have been an oversight.
> 
> Thanks for confirming, I’ll take another pass over the proposed diff in a bit.

Polished a little and pushed to master with a backpatch to 14 where it was
introduced.

--
Daniel Gustafsson		https://vmware.com/

Commits

  1. Fix GRANTED BY support in REVOKE ROLE statements

  2. Allow GRANTED BY clause in normal GRANT and REVOKE statements

  3. Try to stabilize output from rolenames regression test.

  4. Allow CURRENT_ROLE where CURRENT_USER is accepted