rack-accept-header-updater 1.0.1 → 1.1.0

data/VERSION

@@ -1 +1 @@
-1.0.1
+1.1.0

data/lib/rack/accept_header_updater.rb

@@ -34,8 +34,8 @@ module Rack
       unless (ext = ::File.extname(req.path_info)).empty?
         if (mime_type = Rack::Mime::MIME_TYPES[ext.downcase])
           env['HTTP_ACCEPT'] = [mime_type, env['HTTP_ACCEPT']].join(",")
+          req.path_info.gsub!(/#{ext}$/, '')
         end
-        req.path_info.gsub!(/#{ext}$/, '')
       end
       @app.call(env)
     end    

data/rack-accept-header-updater.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{rack-accept-header-updater}
-  s.version = "1.0.1"
+  s.version = "1.1.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Cyril Rohr"]
-  s.date = %q{2009-10-21}
+  s.date = %q{2009-10-22}
   s.description = %q{A Rack middleware for automatically removing file extensions from URIs, and update the Accept HTTP Header accordingly.}
   s.email = %q{cyril.rohr@gmail.com}
   s.extra_rdoc_files = [

data/spec/rack_accept_header_updater_spec.rb

@@ -16,12 +16,6 @@ describe Rack::AcceptHeaderUpdater do
     status, headers, body = Rack::AcceptHeaderUpdater.new(app).call(request)
     body.should == ["application/json,*/*|/some/resource"]
   end
-  it "should remove the file extension even if there are no mime types associated" do
-    app = lambda { |env| [200, {'Content-Type' => 'text/plain'}, [[env['HTTP_ACCEPT'], env['PATH_INFO']].join("|")]] }
-    request = Rack::MockRequest.env_for("/some/resource.json", :input => "foo=bar", 'HTTP_ACCEPT' => '*/*')
-    status, headers, body = Rack::AcceptHeaderUpdater.new(app).call(request)
-    body.should == ["*/*|/some/resource"]
-  end
   it "should be case insensitive for extensions" do
     mime :json, 'application/json'
     app = lambda { |env| [200, {'Content-Type' => 'text/plain'}, [[env['HTTP_ACCEPT'], env['PATH_INFO']].join("|")]] }
@@ -42,5 +36,11 @@ describe Rack::AcceptHeaderUpdater do
     status, headers, body = Rack::AcceptHeaderUpdater.new(app).call(request)
     body.should == ["application/vnd.com.example.Collection+json;level=1,*/*|/some/resource"]
   end
+  it "should not remove the extension if it is not registered in the rack mime types" do
+    app = lambda { |env| [200, {'Content-Type' => 'text/plain'}, [[env['HTTP_ACCEPT'], env['PATH_INFO']].join("|")]] }
+    request = Rack::MockRequest.env_for("/some/resource.x.y", :input => "foo=bar", 'HTTP_ACCEPT' => '*/*')
+    status, headers, body = Rack::AcceptHeaderUpdater.new(app).call(request)
+    body.should == ["*/*|/some/resource.x.y"]
+  end
 end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: rack-accept-header-updater
 version: !ruby/object:Gem::Version 
-  version: 1.0.1
+  version: 1.1.0
 platform: ruby
 authors: 
 - Cyril Rohr
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2009-10-21 00:00:00 +02:00
+date: 2009-10-22 00:00:00 +02:00
 default_executable: 
 dependencies: []