You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@phoenix.apache.org by ma...@apache.org on 2016/09/20 18:05:40 UTC
[17/47] 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/c3cdb2af
Tree: http://git-wip-us.apache.org/repos/asf/phoenix/tree/c3cdb2af
Diff: http://git-wip-us.apache.org/repos/asf/phoenix/diff/c3cdb2af
Branch: refs/heads/calcite
Commit: c3cdb2afbe60a4959237a6ac264f42322a049d1d
Parents: 5a6496e
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:03:43 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/c3cdb2af/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