Horton

Re: Allow CURRENT_ROLE in GRANTED BY

Peter Eisentraut <peter.eisentraut@2ndquadrant.com>

View thread View on postgr.es
From: Peter Eisentraut <peter.eisentraut@2ndquadrant.com>
To: Vik Fearing <vik@postgresfriends.org>, pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2020-12-10T18:39:49Z
Lists: pgsql-hackers

Attachments

On 2020-06-24 20:21, Peter Eisentraut wrote:
> On 2020-06-24 10:12, Vik Fearing wrote:
>> On 6/24/20 8:35 AM, Peter Eisentraut wrote:
>>> I was checking some loose ends in SQL conformance, when I noticed: We
>>> support GRANT role ... GRANTED BY CURRENT_USER, but we don't support
>>> CURRENT_ROLE in that place, even though in PostgreSQL they are
>>> equivalent.  Here is a trivial patch to add that.
>>
>>
>> The only thing that isn't dead-obvious about this patch is the commit
>> message says "[PATCH 1/2]".  What is in the other part?
> 
> Hehe.  The second patch is some in-progress work to add the GRANTED BY
> clause to the regular GRANT command.  More on that perhaps at a later date.

Here is the highly anticipated and quite underwhelming second part of 
this patch set.

-- 
Peter Eisentraut
2ndQuadrant, an EDB company
https://www.2ndquadrant.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