Fix a problem in mod_proxy: it would not set the number of bytes

transferred, so other modules could not access the value from
the request_rec->bytes_sent field.
(And indeed, I also observed some time ago that the bytes_sent field was not set.)

PR: 6841
Submitted by:	Anthony Howe <achowe (at) snert.com>
Reviewed by:	Martin Kraemer

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/1.3.x@95362 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/src/CHANGES b/src/CHANGES
index 076285363603cbcf80f9d911c9f1e219f30a97a6..633b4af8c4126502772953c3514afdc26b4cf063 100644 (file)
--- a/src/CHANGES
+++ b/src/CHANGES
@@ -1,5 +1,10 @@
 Changes with Apache 1.3.25
 
+  *) Fix a problem in mod_proxy: it would not set the number of bytes
+     transferred, so other modules could not access the value from
+     the request_rec->bytes_sent field.
+     [Anthony Howe <achowe at snert.com>] PR#6841
+
   *) Fix a problem in mod_rewrite which would lead to 400 Bad Request
      responses for rewriting rules which resulted in a local path.
      [Martin Kraemer]

diff --git a/src/modules/proxy/proxy_util.c b/src/modules/proxy/proxy_util.c
index e4b30ee7e752ee2994e16736a5151231a261197a..9bfb15bb236ef11d607e02623441d4321efe166b 100644 (file)
--- a/src/modules/proxy/proxy_util.c
+++ b/src/modules/proxy/proxy_util.c
@@ -693,6 +693,9 @@ long int ap_proxy_send_fb(BUFF *f, request_rec *r, cache_req *c, off_t len, int
     }
 
     ap_kill_timeout(r);
+
+    r->bytes_sent += total_bytes_rcvd;
+
     return total_bytes_rcvd;
 }