[PATCH] remove is_member_of_role() from header, add can_set_role()

From: Joshua Brindle <joshua(dot)brindle(at)crunchydata(dot)com>
To: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: [PATCH] remove is_member_of_role() from header, add can_set_role()
Date: 2021-10-27 16:26:56
Message-ID: CAGB+Vh4enxvLBM_BJweWEO12Q0ySLMBWK9iOLaM7e=V1Y0YadA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

As a follow-on to Conflation of member/privs for predefined roles,
this removes is_member_of_role from the header to dissuade it's use
for privilege checking. Since SET ROLE must use membership rather than
privileges a new, explicitly named can_set_role() function is
exported.

is_member_of_role_nosuper() still exists for the following purposes:
- membership loop checking in user.c
- membership matching for pg_hba.conf in hba.c

Other uses of is_member_of_role_nosuper() should be avoided.

Attachment Content-Type Size
0001-unexport-is_member_of_role-add-can_set_role.patch application/octet-stream 3.1 KB

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Jacob Champion 2021-10-27 16:49:21 Re: allowing "map" for password auth methods with clientcert=verify-full
Previous Message Bharath Rupireddy 2021-10-27 16:26:37 Isn't it better with "autovacuum worker...." instead of "worker took too long to start; canceled" specific to "auto