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/09/05 18:54:26 UTC

svn commit: r1165360 - /tomcat/trunk/java/org/apache/coyote/http11/Http11NioProcessor.java

Author: markt
Date: Mon Sep  5 16:54:25 2011
New Revision: 1165360

URL: http://svn.apache.org/viewvc?rev=1165360&view=rev
Log:
Start timing request processing after the request line has been read and before the request headers are read (aligns with other HTTP connectors)

Modified:
    tomcat/trunk/java/org/apache/coyote/http11/Http11NioProcessor.java

Modified: tomcat/trunk/java/org/apache/coyote/http11/Http11NioProcessor.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http11/Http11NioProcessor.java?rev=1165360&r1=1165359&r2=1165360&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/coyote/http11/Http11NioProcessor.java (original)
+++ tomcat/trunk/java/org/apache/coyote/http11/Http11NioProcessor.java Mon Sep  5 16:54:25 2011
@@ -206,7 +206,7 @@ public class Http11NioProcessor extends 
         inputBuffer.init(socketWrapper, endpoint);
         outputBuffer.init(socketWrapper, endpoint);
 
-        // Error flag
+        // Flags
         error = false;
         keepAlive = true;
         comet = false;
@@ -256,6 +256,7 @@ public class Http11NioProcessor extends 
                     }
                 }
                 if (!endpoint.isPaused()) {
+                    request.setStartTime(System.currentTimeMillis());
                     keptAlive = true;
                     if ( !inputBuffer.parseHeaders() ) {
                         //we've read part of the request, don't recycle it
@@ -264,7 +265,6 @@ public class Http11NioProcessor extends 
                         readComplete = false;
                         break;
                     }
-                    request.setStartTime(System.currentTimeMillis());
                     if (!disableUploadTimeout) { //only for body, not for request headers
                         socketWrapper.getSocket().getIOChannel().socket().setSoTimeout(
                                 connectionUploadTimeout);



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