svn merge -r 1450962:1450963 Merging from trunk to branch-2 to fix HADOOP-9339
git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/branches/branch-2@1450966 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
parent
b775d49f35
commit
f2175b3412
|
@ -1064,6 +1064,9 @@ Release 0.23.7 - UNRELEASED
|
||||||
HADOOP-9303. command manual dfsadmin missing entry for restoreFailedStorage
|
HADOOP-9303. command manual dfsadmin missing entry for restoreFailedStorage
|
||||||
option (Andy Isaacson via tgraves)
|
option (Andy Isaacson via tgraves)
|
||||||
|
|
||||||
|
HADOOP-9339. IPC.Server incorrectly sets UGI auth type (Daryn Sharp by
|
||||||
|
kihwal)
|
||||||
|
|
||||||
Release 0.23.6 - UNRELEASED
|
Release 0.23.6 - UNRELEASED
|
||||||
|
|
||||||
INCOMPATIBLE CHANGES
|
INCOMPATIBLE CHANGES
|
||||||
|
|
|
@ -1511,9 +1511,6 @@ public abstract class Server {
|
||||||
UserGroupInformation protocolUser = ProtoUtil.getUgi(connectionContext);
|
UserGroupInformation protocolUser = ProtoUtil.getUgi(connectionContext);
|
||||||
if (saslServer == null) {
|
if (saslServer == null) {
|
||||||
user = protocolUser;
|
user = protocolUser;
|
||||||
if (user != null) {
|
|
||||||
user.setAuthenticationMethod(AuthMethod.SIMPLE);
|
|
||||||
}
|
|
||||||
} else {
|
} else {
|
||||||
// user is authenticated
|
// user is authenticated
|
||||||
user.setAuthenticationMethod(authMethod);
|
user.setAuthenticationMethod(authMethod);
|
||||||
|
|
|
@ -180,7 +180,7 @@ public class TestDoAsEffectiveUser {
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
|
|
||||||
Assert.assertEquals(PROXY_USER_NAME + " (auth:SIMPLE) via " + REAL_USER_NAME + " (auth:SIMPLE)", retVal);
|
Assert.assertEquals(PROXY_USER_NAME + " (auth:PROXY) via " + REAL_USER_NAME + " (auth:SIMPLE)", retVal);
|
||||||
} catch (Exception e) {
|
} catch (Exception e) {
|
||||||
e.printStackTrace();
|
e.printStackTrace();
|
||||||
Assert.fail();
|
Assert.fail();
|
||||||
|
@ -224,7 +224,7 @@ public class TestDoAsEffectiveUser {
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
|
|
||||||
Assert.assertEquals(PROXY_USER_NAME + " (auth:SIMPLE) via " + REAL_USER_NAME + " (auth:SIMPLE)", retVal);
|
Assert.assertEquals(PROXY_USER_NAME + " (auth:PROXY) via " + REAL_USER_NAME + " (auth:SIMPLE)", retVal);
|
||||||
} catch (Exception e) {
|
} catch (Exception e) {
|
||||||
e.printStackTrace();
|
e.printStackTrace();
|
||||||
Assert.fail();
|
Assert.fail();
|
||||||
|
|
Loading…
Reference in New Issue