This project is archived and is in readonly mode.

#2886 ✓invalid
Kieran P

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

Reported by Kieran P | July 9th, 2009 @ 01:09 AM | in 2.x

redirect_to params.merge(:protocol => 'https')

Pretty simple. But it doesn't work. Somewhere along the lines, the hash gets converted to string keys, so when it reaches rewrite_url, and tries to access options[:protocol] it doesn't exist (only options['protocol'] does). This despite passing in an initial symbol.

Quickest fix I've found is attached in a patch to this ticket.

Comments and changes to this ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

<h2 style="font-size: 14px">Tickets have moved to Github</h2>

The new ticket tracker is available at <a href=""></a>

People watching this ticket


Referenced by