| From: | Nathan Bossart <nathandbossart(at)gmail(dot)com> |
|---|---|
| To: | cca5507 <cca5507(at)qq(dot)com> |
| Cc: | Jacob Champion <jacob(dot)champion(at)enterprisedb(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, preTham <prezza672(at)gmail(dot)com>, Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: Why is_admin_of_role() use ROLERECURSE_MEMBERS rather than ROLERECURSE_PRIVS? |
| Date: | 2026-08-07 22:02:21 |
| Message-ID: | anZV7Z_8X2crYJNi@nathan |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Fri, Aug 07, 2026 at 04:36:20PM -0500, Nathan Bossart wrote:
> At all rates, I think I'm inclined to proceed with something like v4. That
> fixes the error message without changing any other behavior, which is
> probably about the best we can do on the back-branches. Perhaps we should
> consider changing is_admin_of_role() to use ROLERECURSE_PRIVS in v20, but I
> don't sense much appetite for that, so... maybe we leave that thread loose
> for now.
>
> If nobody objects, I'll take care of committing/back-patching, hopefully in
> the near future.
Concretely, like the attached.
--
nathan
| Attachment | Content-Type | Size |
|---|---|---|
| v5-0001-Fix-authorization-check-for-role-membership-chang.patch | text/plain | 6.7 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Michael Paquier | 2026-08-07 22:11:42 | Re: Remove unused vacuum-truncate-auto injection point |
| Previous Message | Sami Imseih | 2026-08-07 22:00:43 | Remove unused vacuum-truncate-auto injection point |