Don't add content-length header.

rack · Jun 11, 2024 · e662408 · e662408
1 parent 35b7310
commit e662408
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/lib/rack/test.rb b/lib/rack/test.rb
@@ -363,7 +363,6 @@ def process_request(uri, env)
         close_body(body)
         cookie_jar.merge(last_response.headers['set-cookie'], uri)
         @after_request.each(&:call)
-        @last_response.finish
 
         yield @last_response if block_given?
 

diff --git a/spec/fixtures/fake_app.rb b/spec/fixtures/fake_app.rb
@@ -6,9 +6,6 @@ module Test
     class FakeApp
       def call(env)
         _, h, b = res = handle(env)
-        length = 0
-        b.each{|s| length += s.bytesize}
-        h['content-length'] = length.to_s
         h['content-type'] = 'text/html;charset=utf-8'
         res
       end

diff --git a/spec/rack/test_spec.rb b/spec/rack/test_spec.rb
@@ -503,6 +503,11 @@ def closed?
     last_response['content-type'].must_equal 'text/html;charset=utf-8'
   end
 
+  it 'should not add a content-length header' do
+    request '/'
+    last_response['content-length'].must_be_nil
+  end
+
   it 'raises an error if no requests have been issued' do
     proc do
       last_response