Refactored Context#strip_ignore_headers (thanks @nbibler)

rtomayko · Feb 10, 2012 · 12ae92d · 12ae92d
1 parent 0ed9a12
commit 12ae92d
Showing 1 changed file with 1 addition and 6 deletions.
diff --git a/lib/rack/cache/context.rb b/lib/rack/cache/context.rb
@@ -272,12 +272,7 @@ def store(response)
 
  # Remove all ignored response headers before writing to the cache.
  def strip_ignore_headers(response)
- stripped = false
- ignore_headers.each do |name|
- stripped ||= response.headers.delete(name)
- response.headers.delete(name)
- end
- record :ignore if stripped
+ record :ignore if response.headers.reject! { |name, value| ignore_headers.include? name }
  end
 
  def log_error(exception)