Ticket #8438: rewind_request_body.diff

actionpack/test/controller/cgi_test.rb

@@ -68 +68 @@
-    assert_equal({"flamenco"=> "love"}, @request.request_parameters)
-  end
-end
+
+
+class CgiRequestNeedsRewoundTest < BaseCgiTest
+  def test_body_should_be_rewound
+    data = 'foo'
+    fake_cgi = Struct.new(:env_table, :query_string, :stdinput).new(@request_hash, '', StringIO.new(data))
+    fake_cgi.env_table['CONTENT_LENGTH'] = data.length
+    fake_cgi.env_table['CONTENT_TYPE'] = 'application/x-www-form-urlencoded; charset=utf-8'
+    request = ActionController::CgiRequest.new(fake_cgi)
+    request.request_parameters
+    assert_equal(0, request.body.pos)
+  end
+end

actionpack/lib/action_controller/request.rb

@@ -330 +330 @@
-
-        # Only multipart form parsing expects a stream.
-        if strategy && strategy != :multipart_form
-
+
+
+
-        end
-
-        case strategy