Bläddra i källkod

HADOOP-7272. Remove unnecessary security related info logs. Contributed by Suresh Srinivas and Benoy Antony.

git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/branches/branch-0.22@1346236 13f79535-47bb-0310-9956-ffa450edef68
Konstantin Shvachko 13 år sedan
förälder
incheckning
6e97437165
2 ändrade filer med 10 tillägg och 3 borttagningar
  1. 3 0
      common/CHANGES.txt
  2. 7 3
      common/src/java/org/apache/hadoop/ipc/Server.java

+ 3 - 0
common/CHANGES.txt

@@ -37,6 +37,9 @@ Release 0.22.1 - Unreleased
     HADOOP-7621. Alfredo config should be in a file not readable by users.
     (Aaron T. Myers and Benoy Antony via shv)
 
+    HADOOP-7272. Remove unnecessary security related info logs.
+    (Suresh Srinivas and Benoy Antony via shv)
+
 Release 0.22.0 - 2011-11-29
 
   INCOMPATIBLE CHANGES

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

@@ -1079,12 +1079,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.debug("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.debug("SASL server successfully authenticated client: " + user);
+          }
           rpcMetrics.authenticationSuccesses.inc();
           AUDITLOG.info(AUTH_SUCCESSFULL_FOR + user);
           saslContextEstablished = true;