You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2011/05/09 15:21:27 UTC

svn commit: r1101018 - /tomcat/trunk/java/org/apache/catalina/valves/RemoteIpValve.java

Author: markt
Date: Mon May  9 13:21:26 2011
New Revision: 1101018

URL: http://svn.apache.org/viewvc?rev=1101018&view=rev
Log:
Review from kkolinko

Modified:
    tomcat/trunk/java/org/apache/catalina/valves/RemoteIpValve.java

Modified: tomcat/trunk/java/org/apache/catalina/valves/RemoteIpValve.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/valves/RemoteIpValve.java?rev=1101018&r1=1101017&r2=1101018&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/valves/RemoteIpValve.java (original)
+++ tomcat/trunk/java/org/apache/catalina/valves/RemoteIpValve.java Mon May  9 13:21:26 2011
@@ -700,9 +700,11 @@ public class RemoteIpValve extends Valve
                 try {
                     port = Integer.parseInt(portHeaderValue);
                 } catch (NumberFormatException nfe) {
-                    log.debug(sm.getString(
-                            "remoteIpValve.invalidPortHeader",
-                            portHeaderValue, portHeader), nfe);
+                    if (log.isDebugEnabled()) {
+                        log.debug(sm.getString(
+                                "remoteIpValve.invalidPortHeader",
+                                portHeaderValue, portHeader), nfe);
+                    }
                 }
             }
         }



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org