diff --git a/actionpack/CHANGELOG b/actionpack/CHANGELOG index 303a440c177e7..c4e4211c42d50 100644 --- a/actionpack/CHANGELOG +++ b/actionpack/CHANGELOG @@ -1,5 +1,7 @@ *2.3.1 [RC2] (March 5, 2009)* +* Fixed that redirection would just log the options, not the final url (which lead to "Redirected to #") [DHH] + * Added ability to pass in :public => true to fresh_when, stale?, and expires_in to make the request proxy cachable #2095 [Gregg Pollack] * Fixed that passing a custom form builder would be forwarded to nested fields_for calls #2023 [Eloy Duran/Nate Wiger] diff --git a/actionpack/lib/action_controller/base.rb b/actionpack/lib/action_controller/base.rb index 564fdf677ee41..b769a2e67e602 100644 --- a/actionpack/lib/action_controller/base.rb +++ b/actionpack/lib/action_controller/base.rb @@ -1101,7 +1101,6 @@ def redirect_to(options = {}, response_status = {}) #:doc: end response.redirected_to = options - logger.info("Redirected to #{options}") if logger && logger.info? case options # The scheme name consist of a letter followed by any combination of @@ -1124,6 +1123,7 @@ def redirect_to(options = {}, response_status = {}) #:doc: def redirect_to_full_url(url, status) raise DoubleRenderError if performed? + logger.info("Redirected to #{url}") if logger && logger.info? response.redirect(url, interpret_status(status)) @performed_redirect = true end