Merge pull request #1055 from marshall-lee/remove_case_insensitive_accept

Remove case insensitive Accept header support

Author: dblock

lib/grape/http/headers.rb

@@ -20,7 +20,6 @@ module Headers
       HTTP_TRANSFER_ENCODING = 'HTTP_TRANSFER_ENCODING'.freeze
       HTTP_ACCEPT            = 'HTTP_ACCEPT'.freeze
 
-      ACCEPT                 = 'accept'.freeze
       FORMAT                 = 'format'.freeze
     end
   end

lib/grape/middleware/formatter.rb

@@ -11,13 +11,6 @@ def default_options
         }
       end
 
-      def headers
-        env.dup.inject({}) do |h, (k, v)|
-          h[k.to_s.downcase[5..-1]] = v if k.to_s.downcase.start_with?('http_')
-          h
-        end
-      end
-
       def before
         negotiate_content_type
         read_body_input
@@ -133,7 +126,7 @@ def format_from_header
       end
 
       def mime_array
-        accept = headers[Grape::Http::Headers::ACCEPT]
+        accept = env[Grape::Http::Headers::HTTP_ACCEPT]
         return [] unless accept
 
         accept_into_mime_and_quality = %r{

spec/grape/middleware/formatter_spec.rb

@@ -110,11 +110,6 @@ def to_xml
       expect(subject.env['api.format']).to eq(:xml)
     end
 
-    it 'looks for case-indifferent headers' do
-      subject.call('PATH_INFO' => '/info', 'http_accept' => 'application/xml')
-      expect(subject.env['api.format']).to eq(:xml)
-    end
-
     it 'uses quality rankings to determine formats' do
       subject.call('PATH_INFO' => '/info', 'HTTP_ACCEPT' => 'application/json; q=0.3,application/xml; q=1.0')
       expect(subject.env['api.format']).to eq(:xml)