Author: rjung
Date: Mon Feb  6 17:30:37 2012
New Revision: 1241087

URL: http://svn.apache.org/viewvc?rev=1241087&view=rev
Log:
Fix millisecond handling in AccessLogValve when
used with a general SimpleDateFormat time pattern.

We called tidyFormat() but never used the tidied
format.

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

Modified: tomcat/trunk/java/org/apache/catalina/valves/AccessLogValve.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/valves/AccessLogValve.java?rev=1241087&r1=1241086&r2=1241087&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/valves/AccessLogValve.java (original)
+++ tomcat/trunk/java/org/apache/catalina/valves/AccessLogValve.java Mon Feb  6 
17:30:37 2012
@@ -1431,7 +1431,7 @@ public class AccessLogValve extends Valv
          * chars later with the actual milliseconds because that's
          * relatively cheap.
          */
-        private void tidyFormat() {
+        private String tidyFormat(String format) {
             boolean escape = false;
             StringBuilder result = new StringBuilder();
             int len = format.length();
@@ -1448,6 +1448,7 @@ public class AccessLogValve extends Valv
                     escape = !escape;
                 }
             }
+            return result.toString();
         }
 
         protected DateAndTimeElement(String header) {
@@ -1476,7 +1477,7 @@ public class AccessLogValve extends Valv
                     type = formatType.MSEC_FRAC;
                 } else {
                     type = formatType.SDF;
-                    tidyFormat();
+                    format = tidyFormat(format);
                 }
             }
         }



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

Reply via email to