httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Plüm, Rüdiger, VIS <ruediger.pl...@vodafone.com>
Subject AW: svn commit: r378032 - in /httpd/httpd/trunk: CHANGES modules/proxy/mod_proxy_http.c modules/proxy/proxy_util.c
Date Mon, 20 Feb 2006 10:30:02 GMT


>> -----Ursprüngliche Nachricht-----
>> Von: Joe Orton 
>> > 
>> http://mail-archives.apache.org/mod_mbox/httpd-dev/200602.mbox
/ajax/%3c035587A0-586B-4BD7-A4F3-9D76D5B89552@jaguNET.com%3e
>> this is really weird. Maybe another change that happened after r378032?
>
>Maybe Jim's build didn't include mod_ssl?  r378032 is the most recent 
>change to modules/proxy.

Maybe. Got another idea in the meantime. Maybe it is because we use the same
socket to a backend, but we create a new conn_rec each time for this backend
and thus try to reinit the existing ssl connection. Could you please give the
following patch a try?

Index: proxy_util.c
===================================================================
--- proxy_util.c        (revision 379071)
+++ proxy_util.c        (working copy)
@@ -1511,7 +1511,7 @@
         return APR_SUCCESS;
 
     /* deterimine if the connection need to be closed */
-    if (conn->close_on_recycle || conn->close) {
+    if (conn->close_on_recycle || conn->close || conn->is_ssl) {
         apr_pool_t *p = conn->pool;
         apr_pool_clear(conn->pool);
         memset(conn, 0, sizeof(proxy_conn_rec));

It closes a ssl backend connection once it is returned to the connection pool,
such that we have no persistent ssl connections to the backend.
Obviously this is only a workaround.


Regards

Rüdiger

Mime
View raw message