Issue #6973 - Setup Request/Response objects for success with RequestLog

jetty-server/src/main/java/org/eclipse/jetty/server/HttpChannel.java

@@ -872,9 +872,6 @@ public void onCompleted()
         if (LOG.isDebugEnabled())
             LOG.debug("onCompleted for {} written={}", getRequest().getRequestURI(), getBytesWritten());
 
-        if (_requestLog != null)
-            _requestLog.log(_request, _response);
-
         long idleTO = _configuration.getIdleTimeout();
         if (idleTO >= 0 && getIdleTimeout() != _oldIdleTimeout)
             setIdleTimeout(_oldIdleTimeout);

jetty-server/src/main/java/org/eclipse/jetty/server/Request.java

@@ -1420,6 +1420,29 @@ public String changeSessionId()
      */
     public void onCompleted()
     {
+        HttpChannel httpChannel = getHttpChannel();
+        // httpChannel can be null in some scenarios
+        // it's not possible to use requestlog in those scenarios anyway.
+        if (httpChannel != null)
+        {
+            RequestLog requestLog = httpChannel.getRequestLog();
+            if (requestLog != null)
+            {
+                // Don't allow pulling more parameters
+                _contentParamsExtracted = true;
+
+                // Reset the status code to what was committed
+                MetaData.Response committedResponse = getResponse().getCommittedMetaData();
+                if (committedResponse != null)
+                {
+                    getResponse().setStatus(committedResponse.getStatus());
+                    // TODO: Reset the response headers to what they were when committed
+                }
+
+                requestLog.log(this, getResponse());
+            }
+        }
+
         if (_sessions != null)
         {
             for (Session s:_sessions)