Using redirect_to with params merge of protocol doesn't take effect

Quick update: This is not a problem in Rails 3.0 (just tested on latest master) and wasn't a problem in Rails 2.1. So it seems to only affect the 2.3 branch.

Not reproducible. Following tests passed:

class RedirectController < ActionController::Base
  def redirect_with_protocol
    redirect_to :controller => "things", :action => "stuff", :protocol => "https"
  end
end

class RedirectTest < ActionController::TestCase
  tests RedirectController

  def test_redirect_with_protocol
    get :redirect_with_protocol
    assert_response :redirect
    assert_equal "https://test.host/things/stuff", redirect_to_url
  end
end

Not reproducible - verified Edd Morgan's tests pass on 2-3-stable

(from [00544c778f53b034bf4560548479a20a06d5c22d]) Add test ensuring redirect_to uses the given protocol [#2886 Using redirect_to with params merge of protocol doesn't take effect]

Signed-off-by: Pratik Naik pratiknaik@gmail.com
http://github.com/rails/rails/commit/00544c778f53b034bf4560548479a2...