[Varnish] #700: Response Reason-Phrase can contain SP

[Varnish]

#700: Response Reason-Phrase can contain SP
-------------------+--------------------------------------------------------
 Reporter:  slink  |        Type:  defect
   Status:  new    |    Priority:  normal
Milestone:         |   Component:  build 
  Version:  trunk  |    Severity:  normal
 Keywords:         |  
-------------------+--------------------------------------------------------
 Currently, Varnish refuses a TAB in the reason phrase in a response status
 line.

 My understanding of the rfc is that it should accept a tab.

 I'll send a patch

 http://www.w3.org/Protocols/rfc2616/rfc2616-sec6.html#sec6.1 :

   6.1 Status-Line

   The first line of a Response message is the Status-Line, consisting of
 the protocol version followed by a numeric status code and its associated
 textual phrase, with each element separated by SP characters. No CR or LF
 is allowed except in the final CRLF sequence.

        Status-Line = HTTP-Version SP Status-Code SP Reason-Phrase CRLF

 further down:

       Reason-Phrase  = *<TEXT, excluding CR, LF>

 http://www.w3.org/Protocols/rfc2616/rfc2616-sec2.html#sec2.2 :

        TEXT           = <any OCTET except CTLs,
                         but including LWS>

        LWS            = [CRLF] 1*( SP | HT )

        SP             = <US-ASCII SP, space (32)>
        HT             = <US-ASCII HT, horizontal-tab (9)>

-- 
Ticket URL: <http://www.varnish-cache.org/ticket/700>
Varnish <http://varnish-cache.org/>
The Varnish HTTP Accelerator