From: Bruce Momjian <bruce@momjian.us>
To: mortoozseydi@gmail.com
To: pgsql-docs@lists.postgresql.org
Subject: Re: correct documentation in set role
Date: Tue, 26 Sep 2023 11:44:42 -0400
Message-ID: <ZRL8alby8rOrm3Tu@momjian.us> (raw)
In-Reply-To: <169562797830.3727336.2047897773232975635@wrigleys.postgresql.org>
References: <169562797830.3727336.2047897773232975635@wrigleys.postgresql.org>
On Mon, Sep 25, 2023 at 07:46:18AM +0000, PG Doc comments form wrote:
> The following documentation comment has been logged on the website:
>
> Page: https://www.postgresql.org/docs/16/sql-set-role.html
> Description:
>
> it is good to mention that:
> The specified role_name must be a role that the current session user is a
> member of "and has SET True option on it."
> The second part missed in documentation for set role command.
Good point. How is the attached patch?
--
Bruce Momjian <bruce@momjian.us> https://momjian.us
EDB https://enterprisedb.com
Only you can decide what is important to you.
Attachments:
[text/x-diff] set_role.diff (563B, ../ZRL8alby8rOrm3Tu@momjian.us/2-set_role.diff)
download | inline diff:diff --git a/doc/src/sgml/ref/set_role.sgml b/doc/src/sgml/ref/set_role.sgmlindex 13bad1bf66..a2661945aa 100644--- a/doc/src/sgml/ref/set_role.sgml+++ b/doc/src/sgml/ref/set_role.sgml@@ -42,7 +42,8 @@ RESET ROLE
<para>
The specified <replaceable class="parameter">role_name</replaceable>
- must be a role that the current session user is a member of.+ must be a role that the current session user is a member of and+ is granted <literal>WITH SET TRUE</literal>.
(If the session user is a superuser, any role can be selected.)
</para>
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: pgsql-docs@postgresql.org
Cc: bruce@momjian.us, mortoozseydi@gmail.com, pgsql-docs@lists.postgresql.org
Subject: Re: correct documentation in set role
In-Reply-To: <ZRL8alby8rOrm3Tu@momjian.us>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox