http: ignore Version: header if explicitly set by client
commit7d2295fa774d1c98dfbde2b09d93d58712253d24
authorAugusto Becciu <augusto@jadedpixel.com>
Tue, 8 Jun 2010 02:02:43 +0000 (7 23:02 -0300)
committerEric Wong <normalperson@yhbt.net>
Tue, 8 Jun 2010 02:20:06 +0000 (8 02:20 +0000)
tree10acb007be88b3b07c3433cbd5280a16e18ea56a
parented9cc7ba22bab5b3f7262d869a8bc03f14b2d4cb
http: ignore Version: header if explicitly set by client

When Unicorn receives a request with a "Version" header, the
HttpParser transforms it into "HTTP_VERSION". After that tries to add
it to the request hash which already contains a "HTTP_VERSION" key
with the actual http version of the request. So it tries to append the
new value separated by a comma. But since the http version is a
freezed constant, the TypeError exception is raised.

According to the HTTP RFC
(http://www.w3.org/Protocols/rfc2616/rfc2616-sec7.html#sec7.1) a
"Version" header is valid. However, it's not supported in rack, since
rack has a HTTP_VERSION env variable for the http version. So I think
the easiest way to deal with this problem is to just ignore the header
since it is extremely unusual. We were getting it from a crappy bot.

ref: http://mid.gmane.org/AANLkTimuGgcwNAMcVZdViFWdF-UcW_RGyZAue7phUXps@mail.gmail.com

Acked-by: Eric Wong <normalperson@yhbt.net>
ext/unicorn_http/unicorn_http.rl
test/unit/test_http_parser_ng.rb