Make reason-phrase optional?

[Per Buer]

In the web-forum I noticed someone ran into some trouble with the probes
declaring the backend sick if the backend dropped the reason-phrase (The
"OK" in "HTTP/1.1 200 OK").

According to http://tools.ietf.org/html/rfc2616#section-6.1.1 - the reason
phrase is meant for human consumption and you're allowed to say whatever
you'd like here. The reason phrase is required by the RFC and the question
is whether Varnish should require it or not. After all, it isn't used for
anything so I don't see a reason why we should require it. Then again, we
cannot be faulted for declaring a backend sick when it is in violation of
the protocol.


-- 
 <http://www.varnish-software.com/> *Per Buer*
Varnish Software AS
Phone: +47 958 39 117 | Skype: per.buer
We Make Websites Fly!
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://www.varnish-cache.org/lists/pipermail/varnish-dev/attachments/20130515/9a16c6d5/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: make-reason-optional-for-probe.patch
Type: application/octet-stream
Size: 459 bytes
Desc: not available
URL: <https://www.varnish-cache.org/lists/pipermail/varnish-dev/attachments/20130515/9a16c6d5/attachment.obj>