Apache
/
hbase
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

HBASE-25441 : add security check for some APIs in RSRpcServices (#2832) (#2810) 

Signed-off-by: stack <stack@apache.org>
Signed-off-by: Viraj Jasani <vjasani@apache.org>
This commit is contained in:
lujiefsi 2020-12-31 02:57:57 +08:00 committed by GitHub
parent 14d8f2f268
commit 8a16aed817
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 17 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java
View File

@ -2781,6 +2781,19 @@ public class HMaster extends HRegionServer implements MasterServices, Server {
return initialized.isReady(); return initialized.isReady();
} }
/**
* Report whether this master is started
*
* This method is used for testing.
*
* @return true if master is ready to go, false if not.
*/
@Override
public boolean isOnline() {
return serviceStarted;
}
/** /**
* Report whether this master is in maintenance mode. * Report whether this master is in maintenance mode.
* *

4
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RSRpcServices.java
View File

@ -2224,6 +2224,7 @@ public class RSRpcServices implements HBaseRPCErrorHandler,
@QosPriority(priority=HConstants.ADMIN_QOS) @QosPriority(priority=HConstants.ADMIN_QOS)
public StopServerResponse stopServer(final RpcController controller, public StopServerResponse stopServer(final RpcController controller,
final StopServerRequest request) throws ServiceException { final StopServerRequest request) throws ServiceException {
rpcPreCheck("stopServer");
requestCount.increment(); requestCount.increment();
String reason = request.getReason(); String reason = request.getReason();
regionServer.stop(reason); regionServer.stop(reason);
@ -2233,6 +2234,7 @@ public class RSRpcServices implements HBaseRPCErrorHandler,
@Override @Override
public UpdateFavoredNodesResponse updateFavoredNodes(RpcController controller, public UpdateFavoredNodesResponse updateFavoredNodes(RpcController controller,
UpdateFavoredNodesRequest request) throws ServiceException { UpdateFavoredNodesRequest request) throws ServiceException {
rpcPreCheck("updateFavoredNodes");
List<UpdateFavoredNodesRequest.RegionUpdateInfo> openInfoList = request.getUpdateInfoList(); List<UpdateFavoredNodesRequest.RegionUpdateInfo> openInfoList = request.getUpdateInfoList();
UpdateFavoredNodesResponse.Builder respBuilder = UpdateFavoredNodesResponse.newBuilder(); UpdateFavoredNodesResponse.Builder respBuilder = UpdateFavoredNodesResponse.newBuilder();
for (UpdateFavoredNodesRequest.RegionUpdateInfo regionUpdateInfo : openInfoList) { for (UpdateFavoredNodesRequest.RegionUpdateInfo regionUpdateInfo : openInfoList) {
@ -3334,6 +3336,7 @@ public class RSRpcServices implements HBaseRPCErrorHandler,
RpcController controller, UpdateConfigurationRequest request) RpcController controller, UpdateConfigurationRequest request)
throws ServiceException { throws ServiceException {
try { try {
requirePermission("updateConfiguration", Permission.Action.ADMIN);
this.regionServer.updateConfiguration(); this.regionServer.updateConfiguration();
} catch (Exception e) { } catch (Exception e) {
throw new ServiceException(e); throw new ServiceException(e);
@ -3361,6 +3364,7 @@ public class RSRpcServices implements HBaseRPCErrorHandler,
@QosPriority(priority=HConstants.ADMIN_QOS) @QosPriority(priority=HConstants.ADMIN_QOS)
public ClearSlowLogResponses clearSlowLogsResponses(RpcController controller, public ClearSlowLogResponses clearSlowLogsResponses(RpcController controller,
ClearSlowLogResponseRequest request) throws ServiceException { ClearSlowLogResponseRequest request) throws ServiceException {
rpcPreCheck("clearSlowLogsResponses");
final NamedQueueRecorder namedQueueRecorder = final NamedQueueRecorder namedQueueRecorder =
this.regionServer.getNamedQueueRecorder(); this.regionServer.getNamedQueueRecorder();
boolean slowLogsCleaned = false; boolean slowLogsCleaned = false;