Repository: hbase Updated Branches: refs/heads/branch-1 5f1691d06 -> 9d1deeec8
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/9d1deeec Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/9d1deeec Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/9d1deeec Branch: refs/heads/branch-1 Commit: 9d1deeec8c00d86d097d3b4d00d07ff98266ab19 Parents: 5f1691d Author: lv zehui <[email protected]> Authored: Sat Apr 22 21:20:00 2017 +0800 Committer: Sean Busbey <[email protected]> Committed: Mon Apr 24 13:21:35 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/9d1deeec/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 f3efd6f..90b9c5c 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 @@ -326,6 +326,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")) {
