changeset 4116:9baf12ea5ca1

Upstream: separate pool for peer connections. This is required to support persistent https connections as various ssl structures are allocated from connection's pool.
author Maxim Dounin <mdounin@mdounin.ru>
date Thu, 15 Sep 2011 18:21:24 +0000
parents 70a45a504905
children 103b0d9afe07
files src/http/ngx_http_upstream.c
diffstat 1 files changed, 15 insertions(+), 1 deletions(-) [+]
line wrap: on
line diff
--- a/src/http/ngx_http_upstream.c	Thu Sep 15 18:12:58 2011 +0000
+++ b/src/http/ngx_http_upstream.c	Thu Sep 15 18:21:24 2011 +0000
@@ -1136,8 +1136,20 @@
     c->sendfile &= r->connection->sendfile;
     u->output.sendfile = c->sendfile;
 
-    c->pool = r->pool;
+    if (c->pool == NULL) {
+
+        /* we need separate pool here to be able to cache SSL connections */
+
+        c->pool = ngx_create_pool(128, r->connection->log);
+        if (c->pool == NULL) {
+            ngx_http_upstream_finalize_request(r, u,
+                                               NGX_HTTP_INTERNAL_SERVER_ERROR);
+            return;
+        }
+    }
+
     c->log = r->connection->log;
+    c->pool->log = c->log;
     c->read->log = c->log;
     c->write->log = c->log;
 
@@ -2890,6 +2902,7 @@
         }
 #endif
 
+        ngx_destroy_pool(u->peer.connection->pool);
         ngx_close_connection(u->peer.connection);
     }
 
@@ -2984,6 +2997,7 @@
                        "close http upstream connection: %d",
                        u->peer.connection->fd);
 
+        ngx_destroy_pool(u->peer.connection->pool);
         ngx_close_connection(u->peer.connection);
     }