Skip to content
  1. Mar 03, 2019
  2. Feb 26, 2019
  3. Feb 23, 2019
  4. Feb 20, 2019
  5. Feb 19, 2019
  6. Feb 17, 2019
  7. Feb 15, 2019
  8. Feb 14, 2019
  9. Feb 13, 2019
  10. Feb 12, 2019
  11. Feb 11, 2019
    • ylavic's avatar
      mod_proxy_http: rework the flushing strategy when forwarding the request body. · abe53b11
      ylavic authored
      Since the forwarding of 100-continue (end to end) in r1836588, we depended on
      reading all of the requested HUGE_STRING_LEN bytes to avoid the flushes, but
      this is a bit fragile.
      
      This commit introduces the new stream_reqbody_read() function which will try a
      nonblocking read first and, if it fails with EAGAIN, will flush on the backend
      side before blocking for the next client side read.
      
      We can then use it in stream_reqbody_{chunked,cl}() to flush client forwarded
      data only when necessary. This both allows "optimal" flushing and simplifies
      code (note that spool_reqbody_cl() also makes use of the new function but not
      its nonblocking/flush functionality, thus only for consistency with the two
      others, simplification and common error handling).
      
      Also, since proxy_http_req_t::flushall/subprocess_env::proxy-flushall are now
      meaningless (and unused) on the backend side, they are renamed respectively to
      prefetch_nonblocking/proxy-prefetch-nonblocking, and solely determine whether
      to prefetch in nonblocking mode or not. These flags were trunk only and may
      not be really useful if we decided to prefetch in nonblocking mode in any case,
      but for 2.4.x the opt-in looks wise.
      
      
      git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1853407 13f79535-47bb-0310-9956-ffa450edef68
      abe53b11