font_assets 0.1.10 → 0.1.11

data/CHANGELOG.md

@@ -1,5 +1,11 @@
 ### master
 
+### 0.1.11 / 2014-02-25
+
+Bug Fixes
+
+* Handle cases when PATH_INFO is nil or blank. Thanks [arunthampi](https://github.com/arunthampi)!
+
 ### 0.1.10 / 2013-07-07
 
 Fixing Regression

data/lib/font_assets/middleware.rb

@@ -58,7 +58,11 @@ module FontAssets
     end
 
     def extension(path)
-      "." + path.split("?").first.split(".").last
+      if path.nil? || path.length == 0
+        nil
+      else
+        "." + path.split("?").first.split(".").last
+      end
     end
 
     def font_asset?(path)

data/lib/font_assets/version.rb

@@ -1,3 +1,3 @@
 module FontAssets
-  VERSION = "0.1.10"
+  VERSION = "0.1.11"
 end

data/spec/middleware_spec.rb

@@ -104,6 +104,36 @@ describe FontAssets::Middleware do
         its(['Content-Type']) { should == 'text/plain' }
       end
     end
+
+    context 'when PATH_INFO is nil' do
+      let(:app) { load_app }
+      let(:response) { request app, '/', 'PATH_INFO' => nil }
+
+      context 'the response headers' do
+        subject { response[1] }
+
+        its(["Access-Control-Allow-Headers"]) { should be_nil }
+        its(["Access-Control-Max-Age"]) { should be_nil }
+        its(['Access-Control-Allow-Methods']) { should be_nil }
+        its(['Access-Control-Allow-Origin']) { should be_nil }
+        its(['Content-Type']) { should == 'text/plain' }
+      end
+    end
+
+    context 'when PATH_INFO is empty string' do
+      let(:app) { load_app }
+      let(:response) { request app, '/', 'PATH_INFO' => ''}
+
+      context 'the response headers' do
+        subject { response[1] }
+
+        its(["Access-Control-Allow-Headers"]) { should be_nil }
+        its(["Access-Control-Max-Age"]) { should be_nil }
+        its(['Access-Control-Allow-Methods']) { should be_nil }
+        its(['Access-Control-Allow-Origin']) { should be_nil }
+        its(['Content-Type']) { should == 'text/plain' }
+      end
+    end
   end
 
   context 'for OPTIONS requests' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: font_assets
 version: !ruby/object:Gem::Version
-  version: 0.1.10
+  version: 0.1.11
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-07-04 00:00:00.000000000 Z
+date: 2014-02-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack