1. 28 Feb, 2012 4 commits
  2. 27 Feb, 2012 8 commits
  3. 24 Feb, 2012 12 commits
  4. 03 Nov, 2011 1 commit
  5. 31 Oct, 2011 3 commits
  6. 28 Oct, 2011 4 commits
  7. 26 Oct, 2011 1 commit
  8. 24 Oct, 2011 1 commit
  9. 21 Oct, 2011 3 commits
  10. 18 Oct, 2011 3 commits
    • Kristian Lyngstol's avatar
      Formally remove error from vcl_deliver VCC · 6d263051
      Kristian Lyngstol authored
      Note that error wasn't actually working in vcl_deliver, and this just puts
      VCC in line with the rest of Varnish. Syntax errors are better than assert
      errors.
      
      Re #1027
      
      I'll leave it for later discussion to see if we close #1027, which is
      technically a feature request now, though a request for a feature we used
      to have (not sure how well it worked).
      6d263051
    • Kristian Lyngstol's avatar
      Ensure ban lurker sleeps 1.0s on failure · 1369592d
      Kristian Lyngstol authored
      As per documentation, the ban lurker sleeps ban_lurker_sleep when it is
      successful, but on failure it should only sleep 1.0s. No point hammering
      the ban list every 0.01s if bans aren't even used.
      
      Fixes #1030
      
      Conflicts:
      
      	bin/varnishd/cache_ban.c
      1369592d
    • Kristian Lyngstol's avatar
      Ensure obj->response is set sensibly for errors · 0e3fd5b2
      Kristian Lyngstol authored
      The http_PutProtocol() and http_PutResponse() would in the case of
      workspace overflow leave the headers as NULL and log a
      SLT_LostHeader. This would make Varnish assert correctly later when
      writing to the wire, as these are mandated by HTTP.
      
      This commit changes them to set the fields to static strings instead
      ("HTTP/1.1" and "Lost Response") when failing to write them to the
      workspace. This leaves enough information to complete the protocol in the
      case of overflow.
      
      The patch also increases the synthetic object's workspace from static
      1024 to param->http_resp_size. This leaves more (and configurable)
      room for manipulating the headers of the synthetic object in
      vcl_error.
      
      This whole thing has been a collaboration between Martin and myself. I'll
      leave it a mystery who wrote what line of code, which part of the comment
      and contributed what to the test-case.
      
      In all fairness, it's not a prefect solution, but a far step closer to one.
      So it sort of, kinda, more or less, for now, until we get a better
      solution:
      
      Fixes: #1031
      
      Conflicts:
      
      	bin/varnishd/cache_http.c
      0e3fd5b2