Repository: atlas
Updated Branches:
  refs/heads/master e4921452e -> b919d3d09


ATLAS-2986: updated AtlasClient to skip doAs query parameter when username is 
same as the logged in user


Project: http://git-wip-us.apache.org/repos/asf/atlas/repo
Commit: http://git-wip-us.apache.org/repos/asf/atlas/commit/b919d3d0
Tree: http://git-wip-us.apache.org/repos/asf/atlas/tree/b919d3d0
Diff: http://git-wip-us.apache.org/repos/asf/atlas/diff/b919d3d0

Branch: refs/heads/master
Commit: b919d3d0964efbe1eda40724545d5fa506100a02
Parents: e492145
Author: Madhan Neethiraj <mad...@apache.org>
Authored: Fri Nov 30 15:43:50 2018 -0800
Committer: Madhan Neethiraj <mad...@apache.org>
Committed: Fri Nov 30 15:43:50 2018 -0800

----------------------------------------------------------------------
 .../main/java/org/apache/atlas/security/SecureClientUtils.java  | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/atlas/blob/b919d3d0/client/common/src/main/java/org/apache/atlas/security/SecureClientUtils.java
----------------------------------------------------------------------
diff --git 
a/client/common/src/main/java/org/apache/atlas/security/SecureClientUtils.java 
b/client/common/src/main/java/org/apache/atlas/security/SecureClientUtils.java
index 550ef5a..aeae300 100644
--- 
a/client/common/src/main/java/org/apache/atlas/security/SecureClientUtils.java
+++ 
b/client/common/src/main/java/org/apache/atlas/security/SecureClientUtils.java
@@ -80,9 +80,10 @@ public class SecureClientUtils {
                     (ugiToUse.getAuthenticationMethod() == 
UserGroupInformation.AuthenticationMethod.PROXY)
                     ? ugiToUse.getRealUser() : ugiToUse;
             LOG.info("Real User: {}, is from ticket cache? {}", actualUgi, 
actualUgi.isLoginTicketBased());
-            if (StringUtils.isEmpty(doAsUser)) {
-                doAsUser = actualUgi.getShortUserName();
+            if (StringUtils.isEmpty(doAsUser) || StringUtils.equals(doAsUser, 
actualUgi.getShortUserName())) {
+                doAsUser = null;
             }
+
             LOG.info("doAsUser: {}", doAsUser);
             final String finalDoAsUser = doAsUser;
             httpURLConnectionFactory = new HttpURLConnectionFactory() {

Reply via email to