git.99rst.org Git - openwrt-packages.git/blob

   1 commit 219f7cb9e3eb061103c3c013a6ecf13d38281247
   2 Author: Kevin Zhu <ip0tcp@gmail.com>
   3 Date:   Tue Jan 7 09:42:55 2020 +0100
   4
   5     BUG/MEDIUM: http-ana: Truncate the response when a redirect rule is applied
   6
   7     When a redirect rule is executed on the response path, we must truncate the
   8     received response. Otherwise, the redirect is appended after the response, which
   9     is sent to the client. So it is obviously a bug because the redirect is not
  10     performed. With bodyless responses, it is the "only" bug. But if the response
  11     has a body, the result may be invalid. If the payload is not fully received yet
  12     when the redirect is performed, an internal error is reported.
  13
  14     It must be backported as far as 1.9.
  15
  16     (cherry picked from commit 96b363963f4a4a63823718966798f177a72936b6)
  17     Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
  18
  19 diff --git a/src/http_ana.c b/src/http_ana.c
  20 index ee00d2c76..268796d2e 100644
  21 --- a/src/http_ana.c
  22 +++ b/src/http_ana.c
  23 @@ -2526,6 +2526,8 @@ int http_apply_redirect_rule(struct redirect_rule *rule, struct stream *s, struc
  24                 close = 1;
  25
  26         htx = htx_from_buf(&res->buf);
  27 +       /* Trim any possible response */
  28 +       channel_htx_truncate(&s->res, htx);
  29         flags = (HTX_SL_F_IS_RESP|HTX_SL_F_VER_11|HTX_SL_F_XFER_LEN|HTX_SL_F_BODYLESS);
  30         sl = htx_add_stline(htx, HTX_BLK_RES_SL, flags, ist("HTTP/1.1"), status, reason);
  31         if (!sl)
  32 @@ -2553,6 +2555,8 @@ int http_apply_redirect_rule(struct redirect_rule *rule, struct stream *s, struc
  33         if (!htx_add_endof(htx, HTX_BLK_EOH) || !htx_add_endof(htx, HTX_BLK_EOM))
  34                 goto fail;
  35
  36 +       htx_to_buf(htx, &res->buf);
  37 +
  38         /* let's log the request time */
  39         s->logs.tv_request = now;
  40