[incubator-druid] branch master updated: Unauthorized sql request should return 403 (#6279)

[fj...@apache.org]

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

fjy pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-druid.git


The following commit(s) were added to refs/heads/master by this push:
     new 743547f  Unauthorized sql request should return 403 (#6279)
743547f is described below

commit 743547fc3bbe6dd68a2df3af46cf8c604458f6b0
Author: Dayue Gao <dy...@gmail.com>
AuthorDate: Sun Sep 2 00:17:18 2018 +0800

    Unauthorized sql request should return 403 (#6279)
---
 sql/src/main/java/org/apache/druid/sql/http/SqlResource.java | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/sql/src/main/java/org/apache/druid/sql/http/SqlResource.java b/sql/src/main/java/org/apache/druid/sql/http/SqlResource.java
index 240d796..0406855 100644
--- a/sql/src/main/java/org/apache/druid/sql/http/SqlResource.java
+++ b/sql/src/main/java/org/apache/druid/sql/http/SqlResource.java
@@ -29,6 +29,7 @@ import org.apache.druid.java.util.common.guava.Yielder;
 import org.apache.druid.java.util.common.guava.Yielders;
 import org.apache.druid.java.util.common.logger.Logger;
 import org.apache.druid.query.QueryInterruptedException;
+import org.apache.druid.server.security.ForbiddenException;
 import org.apache.druid.sql.calcite.planner.Calcites;
 import org.apache.druid.sql.calcite.planner.DruidPlanner;
 import org.apache.druid.sql.calcite.planner.PlannerFactory;
@@ -160,6 +161,9 @@ public class SqlResource
         throw Throwables.propagate(e);
       }
     }
+    catch (ForbiddenException e) {
+      throw e; // let ForbiddenExceptionMapper handle this
+    }
     catch (Exception e) {
       log.warn(e, "Failed to handle query: %s", sqlQuery);
 


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org