On Fri, Oct 8, 2010 at 7:56 PM, Michael Koziarski <[email protected]> wrote:
>> Obviously Rails is escaping some characters and not others, and it
>> isn't using Rack::Utils.escape to do the escaping.
>
> I don't believe this was done deliberately.  A quick glance over the
> git history didn't point anything obvious out.  Do you want to take a
> go at rationalising the escaping code a bit?

After some digging, I can say it was done delibritely:
http://github.com/rails/rails/commit/856d2fd874d72dd9f83204affff4edfef3308361

Prehaps josh can enlighten us as to the reason why the change was
made, since the commit message does not do so.

Reversing that commit and
http://github.com/rails/rails/commit/1ee9b40b18a0bed5bb10a0785f7e2730bac983f6
would probably fix things, if you decide to go that route.

Jeremy

-- 
You received this message because you are subscribed to the Google Groups "Ruby 
on Rails: Core" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/rubyonrails-core?hl=en.

Reply via email to