From fefa76f70fdc75c91f80bddce2df7a8825205962 Mon Sep 17 00:00:00 2001 From: Fujii Masao Date: Thu, 12 Sep 2024 19:33:44 +0900 Subject: Remove old RULE privilege completely. The RULE privilege for tables was removed in v8.2, but for backward compatibility, GRANT/REVOKE and privilege functions like has_table_privilege continued to accept the RULE keyword without any effect. After discussions on pgsql-hackers, it was agreed that this compatibility is no longer needed. Since it's been long enough since the deprecation, we've decided to fully remove support for RULE privilege, so GRANT/REVOKE and privilege functions will no longer accept it. Author: Fujii Masao Reviewed-by: Nathan Bossart Discussion: https://postgr.es/m/976a3581-6939-457f-b947-fc3dc836c083@oss.nttdata.com --- src/backend/utils/adt/acl.c | 6 ------ 1 file changed, 6 deletions(-) (limited to 'src/backend/utils/adt/acl.c') diff --git a/src/backend/utils/adt/acl.c b/src/backend/utils/adt/acl.c index d7b39140b3d..4ad222b8d10 100644 --- a/src/backend/utils/adt/acl.c +++ b/src/backend/utils/adt/acl.c @@ -341,9 +341,6 @@ aclparse(const char *s, AclItem *aip, Node *escontext) case ACL_MAINTAIN_CHR: read = ACL_MAINTAIN; break; - case 'R': /* ignore old RULE privileges */ - read = 0; - break; default: ereturn(escontext, NULL, (errcode(ERRCODE_INVALID_TEXT_REPRESENTATION), @@ -1639,7 +1636,6 @@ makeaclitem(PG_FUNCTION_ARGS) {"SET", ACL_SET}, {"ALTER SYSTEM", ACL_ALTER_SYSTEM}, {"MAINTAIN", ACL_MAINTAIN}, - {"RULE", 0}, /* ignore old RULE privileges */ {NULL, 0} }; @@ -2063,8 +2059,6 @@ convert_table_priv_string(text *priv_type_text) {"TRIGGER WITH GRANT OPTION", ACL_GRANT_OPTION_FOR(ACL_TRIGGER)}, {"MAINTAIN", ACL_MAINTAIN}, {"MAINTAIN WITH GRANT OPTION", ACL_GRANT_OPTION_FOR(ACL_MAINTAIN)}, - {"RULE", 0}, /* ignore old RULE privileges */ - {"RULE WITH GRANT OPTION", 0}, {NULL, 0} }; -- cgit v1.2.3