Commit cd2e297f authored by Lasse Karstensen's avatar Lasse Karstensen

Reflect new C: close default in vcl_pipe

We now set Connection: Close on all piped backend requests, to avoid some of
the debugging pain seen over the last 2-3 years.

Basic problem is that request backend routing done in vcl_recv would not be
honored for subsequent requests after one was piped.

This was, at least for new users, a rather nasty surprise.

Original change in commit 880c3c2a.
parent 5261bdb4
......@@ -67,12 +67,11 @@ sub vcl_recv {
}
sub vcl_pipe {
# Note that only the first request to the backend will have
# X-Forwarded-For set. If you use X-Forwarded-For and want to
# have it set for all requests, make sure to have:
# set bereq.http.connection = "close";
# here. It is not set by default as it might break some broken web
# applications, like IIS with NTLM authentication.
# By default Connection: close is set on all piped requests, to stop
# connection reuse from sending future requests directly to the
# (potentially) wrong backend. If you do want this to happen, you can undo
# it here.
# unset bereq.http.connection;
return (pipe);
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment