cache_rules 0.1.15 → 0.1.16

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 0.1.16 (2015-04-29)
+
+  * Errors caused by empty HTTP headers. Fixes issue #8
+
 ## 0.1.15 (2015-04-29)
 
   * No max-stale = allow anything. Fixes issue #7

data/cache_rules.gemspec

@@ -6,7 +6,7 @@ require 'date'
 
 Gem::Specification.new do |s|
   s.name        = 'cache_rules'
-  s.version     = '0.1.15'
+  s.version     = '0.1.16'
 
   s.date        = Date.today.to_s
 

data/lib/formatting.rb

@@ -30,7 +30,7 @@ module CacheRules
   def clean
     ->(headers) {
       Array(headers).reject {|key, value|
-        HEADERS_NO_CACHE.include?(key) || helper_is_if_modified_error?(key, value)
+        HEADERS_NO_CACHE.include?(key) || helper_is_if_modified_error?(key, value) || value.nil? || value.empty?
       }
     }
   end

data/test/test_regressions.rb

@@ -35,4 +35,13 @@ class TestRegressions < MiniTest::Test
 
     assert_kind_of TrueClass, no_max_stale
   end
+
+  # https://github.com/aw/CacheRules/issues/8
+  def test_bugfix_8_errors_caused_by_empty_http_headers
+    isnil   = CacheRules.clean.call({'Content-Type'=>nil})
+    isempty = CacheRules.clean.call({'Content-Type'=>''})
+
+    assert_equal isnil,   []
+    assert_equal isempty, []
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: cache_rules
 version: !ruby/object:Gem::Version
-  version: 0.1.15
+  version: 0.1.16
   prerelease: 
 platform: ruby
 authors: