rack-parser 0.1.1 → 0.1.2

data/README.md

@@ -98,13 +98,14 @@ This project came to being because of:
 
 ## External Sources/Documentations
 
-* [Sinatra book contrib](https://github.com/sinatra/sinatra-book-contrib/blob/master/middleware/rack_parser.md) - mini tutorial on using rack-parser (thanks to [Eric Gjertsen](https://github.com/ericgj))
+* [Sinatra recipes](https://github.com/sinatra/sinatra-recipes/blob/master/middleware/rack_parser.md) - mini tutorial on using rack-parser (thanks to [Eric Gjertsen](https://github.com/ericgj))
 
 
 ## Contributors ##
 
 * [Stephen Becker IV](https://github.com/sbeckeriv) - For initial custom error response handling work.
 * [Tom May](https://github.com/tommay) - skip loading post body unless content type is set.
+* [Moonsik Kang](https://github.com/deepblue) - skip rack parser for content types that are not explicitly set.
 
 ## Copyright
 

data/lib/rack/parser.rb

@@ -63,6 +63,7 @@ module Rack
 
     def _call(env)
       content_type = Rack::Request.new(env).media_type
+      return @app.call(env) if !@content_types[content_type] && !@error_responses[content_type]
       body = env[POST_BODY].read if content_type
       return @app.call(env) if (body.respond_to?(:empty?) ? body.empty? : !body) # Send it down the stack immediately
       begin

data/rack-parser.gemspec

@@ -3,7 +3,7 @@ $:.push File.expand_path("../lib", __FILE__)
 
 Gem::Specification.new do |s|
   s.name        = "rack-parser"
-  s.version     = "0.1.1"
+  s.version     = "0.1.2"
   s.authors     = ["Arthur Chiu"]
   s.email       = ["mr.arthur.chiu@gmail.com"]
   s.homepage    = "https://www.github.com/achiu/rack-parser"

data/test/parser_test.rb

@@ -62,7 +62,7 @@ context "Rack::Parser" do
       end
 
       asserts(:status).equals 400
-      asserts(:body).equals "{\"errors\":\"710: unexpected token at 'fuuuuuuuuuu'\"}"
+      asserts(:body).matches %r!{"errors":"\d+: unexpected token at 'fuuuuuuuuuu'"}!
     end
 
     context "with custom default error message" do
@@ -82,4 +82,13 @@ context "Rack::Parser" do
     asserts(:body).matches %r{Hello world}
   end
 
+  context "for get with unconcerned content_type" do
+    setup do
+      post '/post', 'foo=bar', { 'CONTENT_TYPE' => 'application/x-www-form-urlencoded' }
+    end
+
+    asserts(:status).equals 200
+    asserts(:body).equals({'foo' => 'bar'}.inspect)
+  end
+
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rack-parser
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2011-09-08 00:00:00.000000000Z
+date: 2011-10-28 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
-  requirement: &2173199560 !ruby/object:Gem::Requirement
+  requirement: &2152479300 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2173199560
+  version_requirements: *2152479300
 - !ruby/object:Gem::Dependency
   name: multi_json
-  requirement: &2173199120 !ruby/object:Gem::Requirement
+  requirement: &2152478860 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2173199120
+  version_requirements: *2152478860
 - !ruby/object:Gem::Dependency
   name: multi_xml
-  requirement: &2173198700 !ruby/object:Gem::Requirement
+  requirement: &2152478400 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2173198700
+  version_requirements: *2152478400
 - !ruby/object:Gem::Dependency
   name: riot
-  requirement: &2173198280 !ruby/object:Gem::Requirement
+  requirement: &2152477960 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,10 +54,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2173198280
+  version_requirements: *2152477960
 - !ruby/object:Gem::Dependency
   name: rack-test
-  requirement: &2173197860 !ruby/object:Gem::Requirement
+  requirement: &2152477540 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,10 +65,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2173197860
+  version_requirements: *2152477540
 - !ruby/object:Gem::Dependency
   name: json
-  requirement: &2173197440 !ruby/object:Gem::Requirement
+  requirement: &2152477120 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -76,7 +76,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2173197440
+  version_requirements: *2152477120
 description: Rack Middleware for parsing post body data for json, xml and various
   content types
 email:
@@ -114,7 +114,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: rack-parser
-rubygems_version: 1.8.6
+rubygems_version: 1.8.10
 signing_key: 
 specification_version: 3
 summary: Rack Middleware for parsing post body data