]> git.ipfire.org Git - thirdparty/apache/httpd.git/commit
As much as it pains me, seriously, it seems that reviewing the re-backport
authorWilliam A. Rowe Jr <wrowe@apache.org>
Sat, 6 Aug 2005 21:29:05 +0000 (21:29 +0000)
committerWilliam A. Rowe Jr <wrowe@apache.org>
Sat, 6 Aug 2005 21:29:05 +0000 (21:29 +0000)
commitd9552365a9aeac0ec908863e0b09c4bd2939de98
tree2c48174efbb48a5ae032e4ffa58e1f27a8ab3585
parent99f67d424f2f11045d029d5aa1e8371f3e9e456a
  As much as it pains me, seriously, it seems that reviewing the re-backport
  of this code was too illegible for review, so it seems we will need to
  re-review a fresh backport from httpd trunk.

  Vetoed this patch as it does not follow RFC 2616.  Specifically;

   * it does not respect the T-E over the C-L header

   * it introduces edge cases that allow the administrator to override the
     C-L T-E elections, without enough information to prevent splitting the
     request body due to misaligned C-L's

   * it does not attempt to determine if there is a 'small request body'
     to prefer a legitimate C-L when that option might be available to us

  A revised patch combining Jeff's efforts and mine is available at:

    http://people.apache.org/~wrowe/httpd-2.0.54-proxy-request.patch

  for review, which can be applied to httpd-2.0.54 and now, branches/2.0.x,
  which addresses all of the issues cited above.

  Note that I do not consider the correction of protocol.c to strip the C-L
  where T-E is present to fulfill the need for a correct implementation
  in proxy_http.c; modules have too much flexibility to modify the headers
  before they hit this logic in the proxy request body handling.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.0.x@230592 13f79535-47bb-0310-9956-ffa450edef68
CHANGES
STATUS
modules/proxy/proxy_http.c