summary refs log tree commit
diff options
context:
space:
mode:
authorAaron Patterson <aaron.patterson@gmail.com>2016-02-23 07:33:28 -0800
committerAaron Patterson <aaron.patterson@gmail.com>2016-02-23 07:33:28 -0800
commitea2ed578c3687553e540b7c61e11c0855c316c1a (patch)
treebc600b9af1846e854c80221e70355f71da23cddc
parent146c754df01ae768119f2066421b10efd0f0db4f (diff)
parentf2baf7ba76f90d5977abc347fed3a6bb7d32e114 (diff)
downloadrack-ea2ed578c3687553e540b7c61e11c0855c316c1a.tar.gz
Merge pull request #1009 from WojtekKruszewski/missing_empty_params
Missing empty multipart params
-rw-r--r--lib/rack/multipart/parser.rb2
-rw-r--r--test/spec_multipart.rb1
2 files changed, 1 insertions, 2 deletions
diff --git a/lib/rack/multipart/parser.rb b/lib/rack/multipart/parser.rb
index 2ea9cd80..567c868d 100644
--- a/lib/rack/multipart/parser.rb
+++ b/lib/rack/multipart/parser.rb
@@ -100,8 +100,6 @@ module Rack
               # Generic multipart cases, not coming from a form
               data = {:type => content_type,
                       :name => name, :tempfile => body, :head => head}
-            elsif !filename && data.empty?
-              return
             end
 
             yield data
diff --git a/test/spec_multipart.rb b/test/spec_multipart.rb
index 3a9f7b35..80e49ccb 100644
--- a/test/spec_multipart.rb
+++ b/test/spec_multipart.rb
@@ -95,6 +95,7 @@ describe Rack::Multipart do
     env['CONTENT_TYPE'] = "multipart/form-data; boundary=----WebKitFormBoundaryWLHCs9qmcJJoyjKR"
     params = Rack::Multipart.parse_multipart(env)
     params['profile']['bio'].must_include 'hello'
+    params['profile'].keys.must_include 'public_email'
   end
 
   it "reject insanely long boundaries" do