Merge pull request #31 from spyderdfx/feature/GOODS-1246

fix: use correct tag for js

lib/string_tools.rb

@@ -137,7 +137,7 @@ def strip_all_tags_and_entities(string)
     #
     # Returns String
     def strip_tags_leave_br(string)
-      sanitized = Sanitize.fragment(string, remove_contents: %w(style javascript), elements: %w(p ul li br blockquote))
+      sanitized = Sanitize.fragment(string, remove_contents: %w(style script), elements: %w(p ul li br blockquote))
 
       sanitized.gsub!(/<(p|li|blockquote)[^>]*>/, '')
       sanitized.gsub!(%r{<(br /|ul[^>]*|/[^>]*)>}, '<br />')
@@ -182,7 +182,7 @@ def sanitize(str, attrs = {})
           :attributes => attributes,
           :elements => elements,
           :css => {:properties => Sanitize::Config::RELAXED[:css][:properties]},
-          :remove_contents => %w(style javascript),
+          :remove_contents => %w(style script),
           :allow_comments => false,
           :transformers => transformers
         )

lib/string_tools/version.rb

@@ -1,3 +1,3 @@
 module StringTools
-  VERSION = '0.11.0'.freeze
+  VERSION = '0.11.1'.freeze
 end

spec/string_tools_spec.rb

@@ -10,7 +10,7 @@
     end
 
     it 'removes javascript from string' do
-      sanitized_string = described_class.sanitize('test string<javascript>alert("ALERT");</javascript>' )
+      sanitized_string = described_class.sanitize('test string<script>alert("ALERT");</script>')
       expect(sanitized_string).to eq 'test string'
     end