Просмотр исходного кода

HADOOP-7272. Remove unnecessary security related info logs (porting from trunk)

git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/branches/branch-0.20-security@1125139 13f79535-47bb-0310-9956-ffa450edef68
Suresh Srinivas 14 лет назад
Родитель
Сommit
ac1c7a3ca8
2 измененных файлов с 9 добавлено и 3 удалено
  1. 2 0
      CHANGES.txt
  2. 7 3
      src/core/org/apache/hadoop/ipc/Server.java

+ 2 - 0
CHANGES.txt

@@ -24,6 +24,8 @@ Release 0.20.205.0 - unreleased
     MAPREDUCE-2479. Move distributed cache cleanup to a background task,
     backporting MAPREDUCE-1568. (Robert Joseph Evans via cdouglas)
 
+    HADOOP-7272. Remove unnecessary security related info logs. (suresh)
+
 Release 0.20.204.0 - unreleased
 
   BUG FIXES

+ 7 - 3
src/core/org/apache/hadoop/ipc/Server.java

@@ -1037,12 +1037,16 @@ public abstract class Server {
               null);
         }
         if (saslServer.isComplete()) {
-          LOG.info("SASL server context established. Negotiated QoP is "
-              + saslServer.getNegotiatedProperty(Sasl.QOP));
+          if (LOG.isDebugEnabled()) {
+            LOG.info("SASL server context established. Negotiated QoP is "
+                + saslServer.getNegotiatedProperty(Sasl.QOP));
+          }
           String qop = (String) saslServer.getNegotiatedProperty(Sasl.QOP);
           useWrap = qop != null && !"auth".equalsIgnoreCase(qop);
           user = getAuthorizedUgi(saslServer.getAuthorizationID());
-          LOG.info("SASL server successfully authenticated client: " + user);
+          if (LOG.isDebugEnabled()) {
+            LOG.info("SASL server successfully authenticated client: " + user);
+          }
           rpcMetrics.incrAuthenticationSuccesses();
           AUDITLOG.info(AUTH_SUCCESSFULL_FOR + user);
           saslContextEstablished = true;