Merge pull request #175 from bchaney/allow-css-max-width

Allow CSS property: max-width
flavorjones · Nov 25, 2019 · 8747065 · 8747065
2 parents 2767ae3 + 2699b61
commit 8747065
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/lib/loofah/html5/safelist.rb b/lib/loofah/html5/safelist.rb
@@ -573,6 +573,7 @@ module SafeList
                                             "line-height",
                                             "list-style",
                                             "list-style-type",
+                                            "max-width",
                                             "overflow",
                                             "pause",
                                             "pause-after",

diff --git a/test/html5/test_sanitizer.rb b/test/html5/test_sanitizer.rb
@@ -332,6 +332,13 @@ def test_css_function_sanitization_strips_style_attributes_with_unsafe_functions
     assert_match %r/<span><\/span>/, sane.inner_html
   end
 
+  def test_css_max_width
+    html = '<div style="max-width: 100%;"></div>'
+    sane = Nokogiri::HTML(Loofah.scrub_fragment(html, :escape).to_xml)
+    assert_match %r/max-width/, sane.inner_html
+  end
+
+
   def test_issue_90_slow_regex
     skip("timing tests are hard to make pass and have little regression-testing value")