actually, probably should be URI.escape

jamis · Aug 11, 2012 · 97bdb03 · 97bdb03
1 parent 6fb739d
commit 97bdb03
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/lib/rails_autolink/helpers.rb b/lib/rails_autolink/helpers.rb
@@ -108,13 +108,11 @@ def auto_link_urls(text, html_options = {}, options = {})
                 link_text = block_given?? yield(href) : href
                 href = 'http://' + href unless scheme
 
-                href = href.gsub(/"/, "&quot;")
-
                 unless options[:sanitize] == false
                   link_text = sanitize(link_text)
                   href      = sanitize(href)
                 end
-                content_tag(:a, link_text, link_attributes.merge('href' => href), !!options[:sanitize]) + punctuation.reverse.join('')
+                content_tag(:a, link_text, link_attributes.merge('href' => URI.escape(href.to_str)), !!options[:sanitize]) + punctuation.reverse.join('')
               end
             end
           end