Repository: hbase Updated Branches: refs/heads/branch-1.3 26d278818 -> 1b0ebf8d1
HBASE-17514 emit a warning if thrift1 proxy user is configured but hbase.regionserver.thrift.http is not Signed-off-by: Sean Busbey <[email protected]> Project: http://git-wip-us.apache.org/repos/asf/hbase/repo Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/1b0ebf8d Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/1b0ebf8d Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/1b0ebf8d Branch: refs/heads/branch-1.3 Commit: 1b0ebf8d1ef87ffb4e0f6593b937a34871d0cd45 Parents: 26d2788 Author: lv zehui <[email protected]> Authored: Sat Apr 22 21:20:00 2017 +0800 Committer: Sean Busbey <[email protected]> Committed: Mon Apr 24 14:25:57 2017 -0500 ---------------------------------------------------------------------- .../java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java | 5 +++++ 1 file changed, 5 insertions(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hbase/blob/1b0ebf8d/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java ---------------------------------------------------------------------- diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java index d3e8f20..2d98b5a 100644 --- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java +++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServerRunner.java @@ -322,6 +322,11 @@ public class ThriftServerRunner implements Runnable { this.realUser = userProvider.getCurrent().getUGI(); qop = conf.get(THRIFT_QOP_KEY); doAsEnabled = conf.getBoolean(THRIFT_SUPPORT_PROXYUSER, false); + if (doAsEnabled) { + if (!conf.getBoolean(USE_HTTP_CONF_KEY, false)) { + LOG.warn("Fail to enable the doAs feature. hbase.regionserver.thrift.http is not configured "); + } + } if (qop != null) { if (!qop.equals("auth") && !qop.equals("auth-int") && !qop.equals("auth-conf")) {
