History

Jacques Distler 9e909d5be3 Update Rails, rails_xss and Bundler Update Bundler to 1.0.15. Update Rails to 2.3.12. Update rails_xss plugin. The latter two were the source of a considerable amount of grief, as rails_xss is now MUCH stricter about what string methods can be used. Also made it possible to use rake 0.9.x with Instiki. But you probably REALLY want to use ruby bundle exec rake ... instead of just saying rake ....		2011-06-15 00:43:38 -05:00
..
lib	Update Rails, rails_xss and Bundler	2011-06-15 00:43:38 -05:00
test	Update Rails, rails_xss and Bundler	2011-06-15 00:43:38 -05:00
MIT-LICENSE	Rails_xss Plugin	2010-05-26 00:27:49 -05:00
README.markdown	Rails_xss Plugin	2010-05-26 00:27:49 -05:00
Rakefile	Rails_xss Plugin	2010-05-26 00:27:49 -05:00
init.rb	Update Rails, rails_xss and Bundler	2011-06-15 00:43:38 -05:00

README.markdown

RailsXss

This plugin replaces the default ERB template handlers with erubis, and switches the behaviour to escape by default rather than requiring you to escape. This is consistent with the behaviour in Rails 3.0.

Strings now have a notion of "html safe", which is false by default. Whenever rails copies a string into the response body it checks whether or not the string is safe, safe strings are copied verbatim into the response body, but unsafe strings are escaped first.

All the XSS-proof helpers like link_to and form_tag now return safe strings, and will continue to work unmodified. If you have your own helpers which return strings you know are safe, you will need to explicitly tell rails that they're safe. For an example, take the following helper.

def some_helper
  (1..5).map do |i|
    "<li>#{i}</li>"
  end.join("\n")
end

With this plugin installed, the html will be escaped. So you will need to do one of the following:

Use the raw helper in your template. raw will ensure that your string is copied verbatim into the response body.

<%= raw some_helper %>
Mark the string as safe in the helper itself:

def some_helper (1..5).map do |i| "
#{i}

Use the safe_helper meta programming method:

module ApplicationHelper def some_helper #... end safe_helper :some_helper end

Example

BEFORE:

<%= params[:own_me] %>        => XSS attack
<%=h params[:own_me] %>       => No XSS
<%= @blog_post.content %>     => Displays the HTML

AFTER:

<%= params[:own_me] %>        => No XSS 
<%=h params[:own_me] %>       => No XSS (same result)
<%= @blog_post.content %>     => *escapes* the HTML
<%= raw @blog_post.content %> => Displays the HTML

Gotchas

textilize and simple_format do not return safe strings

Both these methods support arbitrary HTML and are not safe to embed directly in your document. You'll need to do something like:

<%= sanitize(textilize(@blog_post.content_textile)) %>

Safe strings aren't magic.

Once a string has been marked as safe, the only operations which will maintain that HTML safety are String#<<, String#concat and String#+. All other operations are safety ignorant so it's still probably possible to break your app if you're doing something like

value = something_safe
value.gsub!(/a/, params[:own_me])

Don't do that.

String interpolation won't be safe, even when it 'should' be

value = "#{something_safe}#{something_else_safe}"
value.html_safe? # => false

This is intended functionality and can't be fixed.

Getting Started

Install rails 2.3.8 or higher, or freeze rails from 2-3-stable.
Install erubis (gem install erubis)
Install this plugin (ruby script/plugin install git://github.com/rails/rails_xss.git)
Report anything that breaks.