Merge pull request #994 from travisofthenorth/store-exceptions

Store exceptions in request env

Author: lgebhardt

lib/jsonapi/acts_as_resource_controller.rb

@@ -284,6 +284,9 @@ def handle_exceptions(e)
               }
             end
 
+            # Store exception for other middlewares
+            request.env['action_dispatch.exception'] ||= e
+
             internal_server_error = JSONAPI::Exceptions::InternalServerError.new(e)
             Rails.logger.error { "Internal Server Error: #{e.message} #{e.backtrace.join("\n")}" }
             errors = internal_server_error.errors

test/controllers/controller_test.rb

@@ -104,6 +104,21 @@ def test_whitelist_all_exceptions
     JSONAPI.configuration.whitelist_all_exceptions = original_config
   end
 
+  def test_exception_added_to_request_env
+    original_config = JSONAPI.configuration.whitelist_all_exceptions
+    $PostProcessorRaisesErrors = true
+    refute @request.env['action_dispatch.exception']
+    assert_cacheable_get :index
+    assert @request.env['action_dispatch.exception']
+
+    JSONAPI.configuration.whitelist_all_exceptions = true
+    assert_cacheable_get :index
+    assert @request.env['action_dispatch.exception']
+  ensure
+    $PostProcessorRaisesErrors = false
+    JSONAPI.configuration.whitelist_all_exceptions = original_config
+  end
+
   def test_exception_includes_backtrace_when_enabled
     original_config = JSONAPI.configuration.include_backtraces_in_errors
     $PostProcessorRaisesErrors = true