Fixed that invalid accept headers are (1) not processed by rescue handlers and (2) result in status 500 (no method) when http_codes are defined.

Author: croeck

CHANGELOG.md

@@ -10,6 +10,8 @@ Next Release
 * [#886](https://github.com/intridea/grape/pull/886): Group of parameters made to require an explicit type of Hash or Array - [@jrichter1](https://github.com/jrichter1).
 * [#912](https://github.com/intridea/grape/pull/912): Extended the `:using` feature for param documentation to `optional` fields - [@croeck](https://github.com/croeck).
 * [#906](https://github.com/intridea/grape/pull/906): Fix: invalid body parse errors are not rescued by handlers - [@croeck](https://github.com/croeck).
+* [#913](https://github.com/intridea/grape/pull/913): Fix: Invalid accept headers are not processed by rescue handlers - [@croeck](https://github.com/croeck).
+* [#913](https://github.com/intridea/grape/pull/913): Fix: Invalid accept headers cause internal processing errors (500) when http_codes are defined - [@croeck](https://github.com/croeck).
 * Your contribution here.
 
 0.10.1 (12/28/2014)

lib/grape.rb

@@ -58,6 +58,7 @@ module Exceptions
     autoload :MissingGroupTypeError,          'grape/exceptions/missing_group_type'
     autoload :UnsupportedGroupTypeError,      'grape/exceptions/unsupported_group_type'
     autoload :InvalidMessageBody,             'grape/exceptions/invalid_message_body'
+    autoload :InvalidAcceptHeader,            'grape/exceptions/invalid_accept_header'
   end
 
   module ErrorFormatter

lib/grape/error_formatter/base.rb

@@ -41,7 +41,7 @@ def present(message, env)
           http_codes = env['rack.routing_args'][:route_info].route_http_codes || []
           found_code = http_codes.find do |http_code|
             (http_code[0].to_i == env['api.endpoint'].status) && http_code[2].respond_to?(:represent)
-          end
+          end if env['api.endpoint'].request
 
           presenter = found_code[2] if found_code
         end

lib/grape/exceptions/invalid_accept_header.rb

@@ -0,0 +1,10 @@
+# encoding: utf-8
+module Grape
+  module Exceptions
+    class InvalidAcceptHeader < Base
+      def initialize(message, headers)
+        super(message: compose_message('invalid_accept_header', message: message), status: 406, headers: headers)
+      end
+    end
+  end
+end

lib/grape/locale/en.yml

@@ -41,4 +41,7 @@ en:
             "when specifying %{body_format} as content-type, you must pass valid
             %{body_format} in the request's 'body'
             "
+        invalid_accept_header:
+          problem: 'Invalid accept header'
+          resolution: '%{message}'
 

lib/grape/middleware/versioner/header.rb

@@ -23,24 +23,21 @@ module Versioner
       # route.
       class Header < Base
         def before
-          begin
-            header = Rack::Accept::MediaType.new env['HTTP_ACCEPT']
-          rescue RuntimeError => e
-            throw :error, status: 406, headers: error_headers, message: e.message
-          end
+          header = rack_accept_header
 
           if strict?
             # If no Accept header:
             if header.qvalues.empty?
-              throw :error, status: 406, headers: error_headers, message: 'Accept header must be set.'
+              fail Grape::Exceptions::InvalidAcceptHeader.new('Accept header must be set.', error_headers)
             end
             # Remove any acceptable content types with ranges.
             header.qvalues.reject! do |media_type, _|
               Rack::Accept::Header.parse_media_type(media_type).find { |s| s == '*' }
             end
             # If all Accept headers included a range:
             if header.qvalues.empty?
-              throw :error, status: 406, headers: error_headers, message: 'Accept header must not contain ranges ("*").'
+              fail Grape::Exceptions::InvalidAcceptHeader.new('Accept header must not contain ranges ("*").',
+                                                              error_headers)
             end
           end
 
@@ -58,10 +55,10 @@ def before
             end
           # If none of the available content types are acceptable:
           elsif strict?
-            throw :error, status: 406, headers: error_headers, message: '406 Not Acceptable'
+            fail Grape::Exceptions::InvalidAcceptHeader.new('406 Not Acceptable', error_headers)
           # If all acceptable content types specify a vendor or version that doesn't exist:
           elsif header.values.all? { |header_value| has_vendor?(header_value) || version?(header_value) }
-            throw :error, status: 406, headers: error_headers, message: 'API vendor or version not found.'
+            fail Grape::Exceptions::InvalidAcceptHeader.new('API vendor or version not found.', error_headers)
           end
         end
 
@@ -83,7 +80,13 @@ def available_media_types
             available_media_types << media_type
           end
 
-          available_media_types = available_media_types.flatten
+          available_media_types.flatten
+        end
+
+        def rack_accept_header
+          Rack::Accept::MediaType.new env['HTTP_ACCEPT']
+        rescue RuntimeError => e
+          raise Grape::Exceptions::InvalidAcceptHeader.new(e.message, error_headers)
         end
 
         def versions