Re: psql: Add role's membership options to the \du+ command

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Pavel Luzanov <p(dot)luzanov(at)postgrespro(dot)ru>
Cc: "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com>, "Jonathan S(dot) Katz" <jkatz(at)postgresql(dot)org>, Robert Haas <robertmhaas(at)gmail(dot)com>, David Zhang <david(dot)zhang(at)highgo(dot)ca>, "pgsql-hackers(at)lists(dot)postgresql(dot)org" <pgsql-hackers(at)lists(dot)postgresql(dot)org>, rmt(at)lists(dot)postgresql(dot)org, horikyota(dot)ntt(at)gmail(dot)com
Subject: Re: psql: Add role's membership options to the \du+ command
Date: 2023-07-13 16:32:40
Message-ID: 2904260.1689265960@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Pavel Luzanov <p(dot)luzanov(at)postgrespro(dot)ru> writes:
> On 13.07.2023 18:01, Tom Lane wrote:
>> That does not seem right. Is it impossible for pam.set_option
>> to be false? Even if it is, should this code assume that?

> For versions before 16, including one role to another automatically
> gives possibility to issue SET ROLE.

Right, -ENOCAFFEINE.

> IMO, the only question is whether it is correct to show IMPLICIT and
> SET options in versions where they are not actually present
> in pg_auth_members, but can be determined.

Hmm, that's definitely a judgment call. You could argue that it's
useful and consistent, but also that it's confusing to somebody
who's not familiar with the new terminology. On balance I'd lean
to showing them, but I won't fight hard for that viewpoint.

regards, tom lane

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Jeff Davis 2023-07-13 16:36:58 Re: MERGE ... RETURNING
Previous Message Matthias van de Meent 2023-07-13 16:28:39 Re: Potential memory leak in contrib/intarray's g_intbig_compress