diff options
author | Jeremy Kemper <jeremy@bitsweat.net> | 2011-10-11 21:01:11 -0700 |
---|---|---|
committer | Jeremy Kemper <jeremy@bitsweat.net> | 2011-10-11 21:01:11 -0700 |
commit | b8bb5f44c8ba02786ed42d04f66641f236ef42c3 (patch) | |
tree | 391299616f1b40420f4bd444d16f53bc364af260 /activeresource/lib/active_resource | |
parent | 027ecd3afbabb678b5cd161a494641c21c90531d (diff) | |
download | rails-b8bb5f44c8ba02786ed42d04f66641f236ef42c3.tar.gz rails-b8bb5f44c8ba02786ed42d04f66641f236ef42c3.tar.bz2 rails-b8bb5f44c8ba02786ed42d04f66641f236ef42c3.zip |
Treat 303 See Other as a redirect response, too
Diffstat (limited to 'activeresource/lib/active_resource')
-rw-r--r-- | activeresource/lib/active_resource/base.rb | 4 | ||||
-rw-r--r-- | activeresource/lib/active_resource/connection.rb | 2 |
2 files changed, 3 insertions, 3 deletions
diff --git a/activeresource/lib/active_resource/base.rb b/activeresource/lib/active_resource/base.rb index 3fb8ce7790..7b8afdca3c 100644 --- a/activeresource/lib/active_resource/base.rb +++ b/activeresource/lib/active_resource/base.rb @@ -170,8 +170,8 @@ module ActiveResource # <tt>404</tt> is just one of the HTTP error response codes that Active Resource will handle with its own exception. The # following HTTP response codes will also result in these exceptions: # - # * 200..399 - Valid response, no exception (other than 301, 302 and 307) - # * 301, 302, 307 - ActiveResource::Redirection + # * 200..399 - Valid response. No exceptions, other than these redirects: + # * 301, 302, 303, 307 - ActiveResource::Redirection # * 400 - ActiveResource::BadRequest # * 401 - ActiveResource::UnauthorizedAccess # * 403 - ActiveResource::ForbiddenAccess diff --git a/activeresource/lib/active_resource/connection.rb b/activeresource/lib/active_resource/connection.rb index 73410c2d82..94839c8c25 100644 --- a/activeresource/lib/active_resource/connection.rb +++ b/activeresource/lib/active_resource/connection.rb @@ -122,7 +122,7 @@ module ActiveResource # Handles response and error codes from the remote service. def handle_response(response) case response.code.to_i - when 301,302,307 + when 301, 302, 303, 307 raise(Redirection.new(response)) when 200...400 response |