A patch so that http status codes are still included in logs even dur…

…ing an exception [#6333 state:resolved]

Signed-off-by: José Valim <jose.valim@gmail.com>

actionpack/lib/action_controller/log_subscriber.rb

@@ -16,7 +16,11 @@ def process_action(event)
       payload   = event.payload
       additions = ActionController::Base.log_process_action(payload)
 
-      message = "Completed #{payload[:status]} #{Rack::Utils::HTTP_STATUS_CODES[payload[:status]]} in %.0fms" % event.duration
+      status = payload[:status]
+      if status.nil? && payload[:exception].present?
+        status = Rack::Utils.status_code(ActionDispatch::ShowExceptions.rescue_responses[payload[:exception].first]) rescue nil 
+      end 
+      message = "Completed #{status} #{Rack::Utils::HTTP_STATUS_CODES[status]} in %.0fms" % event.duration
       message << " (#{additions.join(" | ")})" unless additions.blank?
 
       info(message)

actionpack/test/controller/log_subscriber_test.rb

@@ -32,6 +32,11 @@ def with_page_cache
       cache_page("Super soaker", "/index.html")
       render :nothing => true
     end
+
+    def with_exception
+      raise Exception
+    end
+
   end
 end
 
@@ -168,6 +173,16 @@ def test_with_page_cache
   ensure
     @controller.config.perform_caching = true
   end
+
+  def test_process_action_with_exception_includes_http_status_code
+    begin
+     get :with_exception
+     wait
+   rescue Exception => e      
+   end
+   assert_equal 2, logs.size
+   assert_match(/Completed 500/, logs.last)
+  end
 
   def logs
     @logs ||= @logger.logged(:info)