]> granicus.if.org Git - apache/commitdiff
mod_http2: fix for latest mod_reqtimeout change re handshake timeouts
authorStefan Eissing <icing@apache.org>
Wed, 20 Feb 2019 15:54:26 +0000 (15:54 +0000)
committerStefan Eissing <icing@apache.org>
Wed, 20 Feb 2019 15:54:26 +0000 (15:54 +0000)
git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1853967 13f79535-47bb-0310-9956-ffa450edef68

modules/http2/h2_mplx.c
modules/http2/h2_task.c

index b828442976612bcd199b1923cb9568f206d661e2..932c04e9b4e4179d4b469b2ddc0b09b390c06fac 100644 (file)
@@ -321,7 +321,7 @@ static int stream_destroy_iter(void *ctx, void *val)
                                && !task->rst_error);
             }
             
-            if (reuse_slave && slave->keepalive == AP_CONN_KEEPALIVE) {
+            if (reuse_slave) {
                 h2_beam_log(task->output.beam, m->c, APLOG_DEBUG, 
                             APLOGNO(03385) "h2_task_destroy, reuse slave");    
                 h2_task_destroy(task);
index 616d57299a2555a21b6c2ba8fceb7df39e8d5f47..77bcb20d8ccb17607b3c5e9eae0138ca2452c871 100644 (file)
@@ -310,11 +310,9 @@ static apr_status_t h2_filter_slave_in(ap_filter_t* f,
         }
     }
     
-    /* Nothing there, no more data to get. Return APR_EAGAIN on
-     * speculative reads, this is ap_check_pipeline()'s trick to
-     * see if the connection needs closing. */
+    /* Nothing there, no more data to get. Return. */
     if (status == APR_EOF && APR_BRIGADE_EMPTY(task->input.bb)) {
-        return (mode == AP_MODE_SPECULATIVE)? APR_EAGAIN : APR_EOF;
+        return status;
     }
 
     if (trace1) {
@@ -600,7 +598,6 @@ apr_status_t h2_task_do(h2_task *task, apr_thread_t *thread, int worker_id)
             slave_id = worker_id; 
         }
         task->c->id = (c->master->id << free_bits)^slave_id;
-        c->keepalive = AP_CONN_KEEPALIVE;
     }
         
     h2_beam_create(&task->output.beam, c->pool, task->stream_id, "output",