From 91150f18c4c7fe6ee912bb835ecaea001c322d30 Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Sun, 9 Aug 2009 03:47:25 -0700 Subject: http: add test for invalid trailer Just in case clients decide to get cute. --- test/unit/test_http_parser_ng.rb | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/test/unit/test_http_parser_ng.rb b/test/unit/test_http_parser_ng.rb index b91013d..6fc0bda 100644 --- a/test/unit/test_http_parser_ng.rb +++ b/test/unit/test_http_parser_ng.rb @@ -236,4 +236,20 @@ class HttpParserNgTest < Test::Unit::TestCase assert_raise(HttpParserError) { @parser.headers({}, str) } end + def test_bad_trailers + str = "PUT / HTTP/1.1\r\n" \ + "Trailer: Transfer-Encoding\r\n" \ + "transfer-Encoding: chunked\r\n\r\n" \ + "1\r\na\r\n2\r\n..\r\n0\r\n" + req = {} + assert_equal req, @parser.headers(req, str) + assert_equal 'Transfer-Encoding', req['HTTP_TRAILER'] + tmp = '' + assert_nil @parser.read_body(tmp, str) + assert_equal 'a..', tmp + assert_equal '', str + str << "Transfer-Encoding: identity\r\n\r\n" + assert_raise(HttpParserError) { @parser.trailers(req, str) } + end + end -- cgit v1.2.3-24-ge0c7