Merge pull request #15012 from schneems/schneems/headers-key

HTTP::Headers#key? correctly converts
rails · May 7, 2014 · 4214063 · 4214063
1 parent cd9222a
commit 4214063
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/actionpack/lib/action_dispatch/http/headers.rb b/actionpack/lib/action_dispatch/http/headers.rb
@@ -26,7 +26,9 @@ def []=(key, value)
         @env[env_name(key)] = value
       end
 
-      def key?(key); @env.key? key; end
+      def key?(key)
+        @env.key? env_name(key)
+      end
       alias :include? :key?
 
       def fetch(key, *args, &block)

diff --git a/actionpack/test/dispatch/header_test.rb b/actionpack/test/dispatch/header_test.rb
@@ -55,6 +55,8 @@ class HeaderTest < ActiveSupport::TestCase
   test "key?" do
     assert @headers.key?("CONTENT_TYPE")
     assert @headers.include?("CONTENT_TYPE")
+    assert @headers.key?("Content-Type")
+    assert @headers.include?("Content-Type")
   end
 
   test "fetch with block" do