diff options
| author | Jeremy Evans <code@jeremyevans.net> | 2019-08-11 12:57:11 -0700 |
|---|---|---|
| committer | Hiroshi SHIBATA <hsbt@ruby-lang.org> | 2019-11-30 17:50:13 +0900 |
| commit | a98632d5c20e41e05074384b8f760af407e6d52a (patch) | |
| tree | e4c77657b6db0e30735ff18c77059d2b8ddb9e07 | |
| parent | edfbffb241a894eece57155a22a89637b33cfe58 (diff) | |
[ruby/webrick] Enabled chunked encoding if Transfer-Encoding: chunked header is set
Patch from Leonard Garvey.
Fixes Ruby Bug 9986.
https://github.com/ruby/webrick/commit/8cff7f3995
| -rw-r--r-- | lib/webrick/httpresponse.rb | 1 | ||||
| -rw-r--r-- | test/webrick/test_httpresponse.rb | 7 |
2 files changed, 8 insertions, 0 deletions
diff --git a/lib/webrick/httpresponse.rb b/lib/webrick/httpresponse.rb index 9942a59714..ba4494ab74 100644 --- a/lib/webrick/httpresponse.rb +++ b/lib/webrick/httpresponse.rb @@ -155,6 +155,7 @@ module WEBrick # Sets the response header +field+ to +value+ def []=(field, value) + @chunked = value.to_s.downcase == 'chunked' if field.downcase == 'transfer-encoding' @header[field.downcase] = value.to_s end diff --git a/test/webrick/test_httpresponse.rb b/test/webrick/test_httpresponse.rb index 696163ba18..e8c69a1212 100644 --- a/test/webrick/test_httpresponse.rb +++ b/test/webrick/test_httpresponse.rb @@ -133,6 +133,13 @@ module WEBrick assert_equal 0, logger.messages.length end + def test_200_chunked_does_not_set_content_length + res.chunked = false + res["Transfer-Encoding"] = 'chunked' + res.setup_header + assert_nil res.header.fetch('content-length', nil) + end + def test_send_body_io IO.pipe {|body_r, body_w| body_w.write 'hello' |