From 1cdc511b90eee7351f21359c32c6d13385846dbd Mon Sep 17 00:00:00 2001 From: Timm Date: Fri, 9 Aug 2013 23:18:37 +0200 Subject: Added LinkScrubber to remove duplication in LinkSanitizer. As such made PermitScrubber easier to subclass. --- .../lib/action_view/helpers/sanitize_helper/sanitizers.rb | 12 ++---------- 1 file changed, 2 insertions(+), 10 deletions(-) (limited to 'actionview/lib/action_view/helpers/sanitize_helper/sanitizers.rb') diff --git a/actionview/lib/action_view/helpers/sanitize_helper/sanitizers.rb b/actionview/lib/action_view/helpers/sanitize_helper/sanitizers.rb index 3ba46ccaa9..905fa38446 100644 --- a/actionview/lib/action_view/helpers/sanitize_helper/sanitizers.rb +++ b/actionview/lib/action_view/helpers/sanitize_helper/sanitizers.rb @@ -1,6 +1,6 @@ require 'active_support/core_ext/class/attribute' require 'active_support/deprecation' -require 'action_view/helpers/sanitize_helper/permit_scrubber' +require 'action_view/helpers/sanitize_helper/scrubbers' module ActionView XPATHS_TO_REMOVE = %w{.//script .//form comment()} @@ -34,15 +34,7 @@ module ActionView class LinkSanitizer < Sanitizer def initialize - @strip_tags = %w(a href) - @link_scrubber = Loofah::Scrubber.new do |node| - if @strip_tags.include?(node.name) - node.before node.children - node.remove - else - Loofah::HTML5::Scrub.scrub_attributes(node) - end - end + @link_scrubber = LinkScrubber.new end def sanitize(html, options = {}) -- cgit v1.2.3