You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by st...@apache.org on 2019/03/28 19:25:26 UTC
[hbase] branch master updated: HBASE-22123 REST gateway reports
Insufficient permissions exceptions as 404 Not Found
This is an automated email from the ASF dual-hosted git repository.
stack pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hbase.git
The following commit(s) were added to refs/heads/master by this push:
new 541f223 HBASE-22123 REST gateway reports Insufficient permissions exceptions as 404 Not Found
541f223 is described below
commit 541f223de4a9b3d9c861c1ba11c368fdf666492a
Author: Tomas Sokorai <>
AuthorDate: Thu Mar 28 12:23:49 2019 -0700
HBASE-22123 REST gateway reports Insufficient permissions exceptions as 404 Not Found
---
.../main/java/org/apache/hadoop/hbase/rest/RowResultGenerator.java | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/RowResultGenerator.java b/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/RowResultGenerator.java
index d08204d..9bc3ec3 100644
--- a/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/RowResultGenerator.java
+++ b/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/RowResultGenerator.java
@@ -26,6 +26,7 @@ import java.util.NoSuchElementException;
import org.apache.hadoop.hbase.Cell;
import org.apache.hadoop.hbase.CellUtil;
import org.apache.hadoop.hbase.DoNotRetryIOException;
+import org.apache.hadoop.hbase.security.AccessDeniedException;
import org.apache.hadoop.hbase.client.Get;
import org.apache.hadoop.hbase.client.Result;
import org.apache.hadoop.hbase.client.Table;
@@ -78,6 +79,10 @@ public class RowResultGenerator extends ResultGenerator {
// help to avoid confusion by leaving a record of what happened here in
// the log.
LOG.warn(StringUtils.stringifyException(e));
+ // Lets get the exception rethrown to get a more meaningful error message than 404
+ if (e instanceof AccessDeniedException) {
+ throw e;
+ }
} finally {
table.close();
}