HADOOP-11962. Sasl message with MD5 challenge text shouldn't be LOG out even in debug level. Contributed by Junping Du.

This commit is contained in:
Haohui Mai 2015-05-12 10:30:32 -07:00
parent 5c2f05cd9b
commit 2f4b6d1157
4 changed files with 3 additions and 9 deletions

View File

@ -668,6 +668,9 @@ Release 2.8.0 - UNRELEASED
HADOOP-11947. test-patch should return early from determine-issue when HADOOP-11947. test-patch should return early from determine-issue when
run in jenkins mode. (Sean Busbey via aw) run in jenkins mode. (Sean Busbey via aw)
HADOOP-11962. Sasl message with MD5 challenge text shouldn't be LOG out
even in debug level. (Junping Du via wheat9)
Release 2.7.1 - UNRELEASED Release 2.7.1 - UNRELEASED
INCOMPATIBLE CHANGES INCOMPATIBLE CHANGES

View File

@ -1488,9 +1488,6 @@ private RpcSaslProto buildSaslResponse(SaslState state, byte[] replyToken) {
} }
private void doSaslReply(Message message) throws IOException { private void doSaslReply(Message message) throws IOException {
if (LOG.isDebugEnabled()) {
LOG.debug("Sending sasl message "+message);
}
setupResponse(saslResponse, saslCall, setupResponse(saslResponse, saslCall,
RpcStatusProto.SUCCESS, null, RpcStatusProto.SUCCESS, null,
new RpcResponseWrapper(message), null, null); new RpcResponseWrapper(message), null, null);

View File

@ -385,9 +385,6 @@ public AuthMethod saslConnect(InputStream inS, OutputStream outS)
} }
RpcSaslProto saslMessage = RpcSaslProto saslMessage =
RpcSaslProto.parseFrom(responseWrapper.getMessageBytes()); RpcSaslProto.parseFrom(responseWrapper.getMessageBytes());
if (LOG.isDebugEnabled()) {
LOG.debug("Received SASL message "+saslMessage);
}
// handle sasl negotiation process // handle sasl negotiation process
RpcSaslProto.Builder response = null; RpcSaslProto.Builder response = null;
switch (saslMessage.getState()) { switch (saslMessage.getState()) {

View File

@ -865,9 +865,6 @@ private synchronized KerberosTicket getTGT() {
.getPrivateCredentials(KerberosTicket.class); .getPrivateCredentials(KerberosTicket.class);
for (KerberosTicket ticket : tickets) { for (KerberosTicket ticket : tickets) {
if (SecurityUtil.isOriginalTGT(ticket)) { if (SecurityUtil.isOriginalTGT(ticket)) {
if (LOG.isDebugEnabled()) {
LOG.debug("Found tgt " + ticket);
}
return ticket; return ticket;
} }
} }