]> granicus.if.org Git - apache/commitdiff
Comment on r1656259.
authorYann Ylavic <ylavic@apache.org>
Sun, 19 Apr 2015 22:14:34 +0000 (22:14 +0000)
committerYann Ylavic <ylavic@apache.org>
Sun, 19 Apr 2015 22:14:34 +0000 (22:14 +0000)
git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.4.x@1674703 13f79535-47bb-0310-9956-ffa450edef68

STATUS

diff --git a/STATUS b/STATUS
index 9d1835893616624c3119e8bc57b686f4bcaf1a73..d2a08db0cd75263170b53d7e654491d6b974fd45 100644 (file)
--- a/STATUS
+++ b/STATUS
@@ -230,6 +230,15 @@ PATCHES PROPOSED TO BACKPORT FROM TRUNK:
                the backend is available to rec' it. I understand the
                error described, but is the fix actually worth it (plus
                it seems to allow for a DDoS vector).
+     ylavic: It seems to me that the problem is real since we reuse the
+             connection before prefetching 16K (either controlled by the
+             client, or by an input filter), we currently always prefetch
+             these bytes already. Regarding performance I don't see any
+             difference (more cycles) compared with the current code.
+             However I think I failed to rebuild the header_brigade when
+             the proxy loop is retried (ping), so I need to rework this.
+             Do you think we'd better remove the prefetch, or maybe just
+             make it nonblocking (by default)?
 
   *) http: Make ap_die() robust against any HTTP error code and not modify
      response status (finally logged) when nothing is to be done. PR 56035.