Repository: hbase Updated Branches: refs/heads/0.98 759976675 -> 5f328fef5 refs/heads/branch-1 6ee046264 -> 07d2f160b refs/heads/master 7b7648322 -> 411991cf1
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/13ab01fb Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/13ab01fb Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/13ab01fb Branch: refs/heads/0.98 Commit: 13ab01fbabc75ced317c34971cb4d183a102681a Parents: 7599766 Author: Andrew Purtell <apurt...@apache.org> Authored: Mon Sep 22 17:37:15 2014 -0700 Committer: Andrew Purtell <apurt...@apache.org> Committed: Mon Sep 22 17:37:15 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/13ab01fb/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(); }
