You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@phoenix.apache.org by el...@apache.org on 2016/09/11 19:28:32 UTC
[3/7] phoenix git commit: PHOENIX-3176 Remove unnecessary PQS proxy
user authorization
PHOENIX-3176 Remove unnecessary PQS proxy user authorization
Project: http://git-wip-us.apache.org/repos/asf/phoenix/repo
Commit: http://git-wip-us.apache.org/repos/asf/phoenix/commit/a278455a
Tree: http://git-wip-us.apache.org/repos/asf/phoenix/tree/a278455a
Diff: http://git-wip-us.apache.org/repos/asf/phoenix/diff/a278455a
Branch: refs/heads/4.x-HBase-0.98
Commit: a278455a43ff22dc2834af3ef9a7b49f1efe853f
Parents: 0a5846c
Author: Josh Elser <el...@apache.org>
Authored: Sat Sep 10 16:05:15 2016 -0400
Committer: Josh Elser <el...@apache.org>
Committed: Sun Sep 11 15:19:11 2016 -0400
----------------------------------------------------------------------
.../main/java/org/apache/phoenix/queryserver/server/Main.java | 4 ----
1 file changed, 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/phoenix/blob/a278455a/phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/Main.java
----------------------------------------------------------------------
diff --git a/phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/Main.java b/phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/Main.java
index 4b3ca7e..0ed3b7b 100644
--- a/phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/Main.java
+++ b/phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/Main.java
@@ -286,10 +286,6 @@ public final class Main extends Configured implements Tool, Runnable {
// LoadingCache will create a new instance for us if one isn't cached.
UserGroupInformation proxyUser = createProxyUser(remoteUserName);
- // Check if this user is allowed to be impersonated.
- // Will throw AuthorizationException if the impersonation as this user is not allowed
- ProxyUsers.authorize(proxyUser, remoteAddress);
-
// Execute the actual call as this proxy user
return proxyUser.doAs(new PrivilegedExceptionAction<T>() {
@Override