You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by ni...@apache.org on 2024/09/24 16:34:46 UTC

(hbase) branch branch-2.5 updated: HBASE-28868 Add missing permission check for updateRSGroupConfig in branch-2 (#6277) (#6281)

This is an automated email from the ASF dual-hosted git repository.

nihaljain pushed a commit to branch branch-2.5
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/branch-2.5 by this push:
     new bfe5c88630e HBASE-28868 Add missing permission check for updateRSGroupConfig in branch-2 (#6277) (#6281)
bfe5c88630e is described below

commit bfe5c88630edcef2b3dfdedbf3f19ce8cc79b3ca
Author: Nihal Jain <ni...@apache.org>
AuthorDate: Tue Sep 24 22:04:39 2024 +0530

    HBASE-28868 Add missing permission check for updateRSGroupConfig in branch-2 (#6277) (#6281)
    
    Signed-off-by: Duo Zhang <zh...@apache.org>
    (cherry picked from commit cf9c1d5a0276000d9e2e6f56aed162adc5484a25)
---
 .../main/java/org/apache/hadoop/hbase/rsgroup/RSGroupAdminEndpoint.java  | 1 +
 1 file changed, 1 insertion(+)

diff --git a/hbase-rsgroup/src/main/java/org/apache/hadoop/hbase/rsgroup/RSGroupAdminEndpoint.java b/hbase-rsgroup/src/main/java/org/apache/hadoop/hbase/rsgroup/RSGroupAdminEndpoint.java
index 7b22b8a3061..0d5188ce735 100644
--- a/hbase-rsgroup/src/main/java/org/apache/hadoop/hbase/rsgroup/RSGroupAdminEndpoint.java
+++ b/hbase-rsgroup/src/main/java/org/apache/hadoop/hbase/rsgroup/RSGroupAdminEndpoint.java
@@ -447,6 +447,7 @@ public class RSGroupAdminEndpoint implements MasterCoprocessor, MasterObserver {
         if (master.getMasterCoprocessorHost() != null) {
           master.getMasterCoprocessorHost().preUpdateRSGroupConfig(groupName, configuration);
         }
+        checkPermission("updateRSGroupConfig");
         groupAdminServer.updateRSGroupConfig(groupName, configuration);
         if (master.getMasterCoprocessorHost() != null) {
           master.getMasterCoprocessorHost().postUpdateRSGroupConfig(groupName, configuration);