public inbox for [email protected]  

help / color / mirror / Atom feed

From: Nathan Bossart <[email protected]>
To: Tom Lane <[email protected]>
Cc: Robert Haas <[email protected]>
Cc: Peter Eisentraut <[email protected]>
Cc: David G. Johnston <[email protected]>
Cc: Ing. Marijo Kristo <[email protected]>
Cc: PostgreSQL Bug List <[email protected]>
Subject: Re: Revoke Connect Privilege from Database not working
Date: Wed, 18 Mar 2026 12:47:17 -0500
Message-ID: <abrlJRPovgwR6J0J@nathan> (raw)
In-Reply-To: <[email protected]>
References: <CAKFQuwbB-ZKtN_p_y5sWa2MrTuy5=pRNPWSj1Ud4HHvTuhb54w@mail.gmail.com>
	<[email protected]>
	<CAKFQuwbpC5w6sUq8gZQATrviZUT4bYpxW+=2uH6sWWMg7fWjzg@mail.gmail.com>
	<aRYLkTpazxKhnS_w@nathan>
	<[email protected]>
	<aXDwtbXCu42Fdmrn@nathan>
	<[email protected]>
	<aatOzgie9RlzbGoo@nathan>
	<abn22fc7-oHGObbZ@nathan>
	<[email protected]>

On Wed, Mar 18, 2026 at 12:36:17PM -0400, Tom Lane wrote:
> Thanks!  I was about to conclude that "silence means assent" and
> do that work, but you beat me to it.  Your changes look fine,
> except that where you have

Thanks for reviewing.

> +   ... A role can only attribute a grant
> +   to another role if they possess the privileges of that role.
> 
> the word "possess" seems a little ambiguous --- it's not clear whether
> it means SET or INHERIT privileges.  The grammar nerd in me doesn't
> like "they" either.  How about s/they possess/it inherits/ ?
> (likewise in revoke.sgml)

This crossed my mind when copy/pasting from the "GRANT on Roles" section,
but I obviously didn't do anything about it.  Fixed in the attached.

-- 
nathan

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected]
  Subject: Re: Revoke Connect Privilege from Database not working
  In-Reply-To: <abrlJRPovgwR6J0J@nathan>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox