svn commit: r1346236 - in /hadoop/common/branches/branch-0.22/common: CHANGES.txt src/java/org/apache/hadoop/ipc/Server.java

[shv]

Author: shv
Date: Tue Jun  5 04:28:54 2012
New Revision: 1346236

URL: http://svn.apache.org/viewvc?rev=1346236&view=rev
Log:
HADOOP-7272. Remove unnecessary security related info logs. Contributed by 
Suresh Srinivas and Benoy Antony.

Modified:
    hadoop/common/branches/branch-0.22/common/CHANGES.txt
    
hadoop/common/branches/branch-0.22/common/src/java/org/apache/hadoop/ipc/Server.java

Modified: hadoop/common/branches/branch-0.22/common/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.22/common/CHANGES.txt?rev=1346236&r1=1346235&r2=1346236&view=diff
==============================================================================
--- hadoop/common/branches/branch-0.22/common/CHANGES.txt (original)
+++ hadoop/common/branches/branch-0.22/common/CHANGES.txt Tue Jun  5 04:28:54 
2012
@@ -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

Modified: 
hadoop/common/branches/branch-0.22/common/src/java/org/apache/hadoop/ipc/Server.java
URL: 
http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.22/common/src/java/org/apache/hadoop/ipc/Server.java?rev=1346236&r1=1346235&r2=1346236&view=diff
==============================================================================
--- 
hadoop/common/branches/branch-0.22/common/src/java/org/apache/hadoop/ipc/Server.java
 (original)
+++ 
hadoop/common/branches/branch-0.22/common/src/java/org/apache/hadoop/ipc/Server.java
 Tue Jun  5 04:28:54 2012
@@ -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;