Browse files

Use 400 Bad Request status for unrescued ActiveRecord::RecordInvalid …

…exceptions.

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@5771 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
  • Loading branch information...
1 parent bc3d150 commit a1ca37ec8659800e5620630490ee8fa46bb3a217 @jeremy jeremy committed Dec 22, 2006
View
2 actionpack/CHANGELOG
@@ -1,5 +1,7 @@
*SVN*
+* Use 400 Bad Request status for unrescued ActiveRecord::RecordInvalid exceptions. [Jeremy Kemper]
+
* Silence log_error deprecation warnings from inspecting deprecated instance variables. [Nate Wiger]
* Only cache GET requests with a 200 OK response. #6514, #6743 [RSL, anamba]
View
3 actionpack/lib/action_controller/rescue.rb
@@ -12,7 +12,8 @@ module Rescue
DEFAULT_RESCUE_RESPONSES = {
'ActionController::RoutingError' => :not_found,
'ActionController::UnknownAction' => :not_found,
- 'ActiveRecord::RecordNotFound' => :not_found
+ 'ActiveRecord::RecordNotFound' => :not_found,
+ 'ActiveRecord::RecordInvalid' => :bad_request
}
DEFAULT_RESCUE_TEMPLATE = 'diagnostics'
View
1 actionpack/test/controller/rescue_test.rb
@@ -144,6 +144,7 @@ def test_rescue_responses
assert_equal :not_found, responses[ActionController::RoutingError.name]
assert_equal :not_found, responses[ActionController::UnknownAction.name]
assert_equal :not_found, responses['ActiveRecord::RecordNotFound']
+ assert_equal :bad_request, responses['ActiveRecord::RecordInvalid']
end
def test_rescue_templates

0 comments on commit a1ca37e

Please sign in to comment.