You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by ap...@apache.org on 2014/09/23 02:37:40 UTC

[4/9] git commit: HBASE-12043 REST server should respond with FORBIDDEN(403) code on AccessDeniedException (Aditya Kishore)

HBASE-12043 REST server should respond with FORBIDDEN(403) code on AccessDeniedException (Aditya Kishore)


Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/e2cd9221
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/e2cd9221
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/e2cd9221

Branch: refs/heads/branch-1
Commit: e2cd9221daf0284bc141ce655f973179d2bb990f
Parents: 6ee0462
Author: Andrew Purtell <ap...@apache.org>
Authored: Mon Sep 22 17:37:17 2014 -0700
Committer: Andrew Purtell <ap...@apache.org>
Committed: Mon Sep 22 17:37:17 2014 -0700

----------------------------------------------------------------------
 .../main/java/org/apache/hadoop/hbase/rest/ResourceBase.java  | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/e2cd9221/hbase-server/src/main/java/org/apache/hadoop/hbase/rest/ResourceBase.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/rest/ResourceBase.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/rest/ResourceBase.java
index fd46b31..d333da1 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/rest/ResourceBase.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/rest/ResourceBase.java
@@ -50,8 +50,11 @@ public class ResourceBase implements Constants {
       //some access denied exceptions are buried
       while (curr != null) {
         if(accessDeniedClazz.isAssignableFrom(curr.getClass())) {
-          throw new SecurityException("Unauthorized" + CRLF +
-              StringUtils.stringifyException(exp) + CRLF);
+          throw new WebApplicationException(
+              Response.status(Response.Status.FORBIDDEN)
+                .type(MIMETYPE_TEXT).entity("Forbidden" + CRLF +
+                   StringUtils.stringifyException(exp) + CRLF)
+                .build());
         }
         curr = curr.getCause();
       }