Fix #44

mganss · Sep 14, 2015 · 486be15 · 486be15
1 parent edd0ad4
commit 486be15
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 1 deletion.
diff --git a/HtmlSanitizer.Tests/Tests.cs b/HtmlSanitizer.Tests/Tests.cs
@@ -2263,6 +2263,23 @@ public void MultipleRecipientsTest()
             var expected = @"<a>Bang Bang</a>";
             Assert.That(actual, Is.EqualTo(expected).IgnoreCase);
         }
+
+        [Test]
+        public void QuotedBackgroundImageTest()
+        {
+            // https://github.com/mganss/HtmlSanitizer/issues/44
+
+            // Arrange
+            var s = new HtmlSanitizer();
+
+            // Act
+            var htmlFragment = "<div style=\"background-image: url('some/random/url.img')\"></div>";
+            var actual = s.Sanitize(htmlFragment);
+
+            // Assert
+            var expected = "<div style=\"background-image: url('some/random/url.img')\"></div>";
+            Assert.That(actual, Is.EqualTo(expected).IgnoreCase);
+        }
     }
 }
 

diff --git a/HtmlSanitizer/HtmlSanitizer.cs b/HtmlSanitizer/HtmlSanitizer.cs
@@ -418,7 +418,7 @@ private bool IsAllowedAttribute(KeyValuePair<string, string> attribute)
         private static readonly Regex CssComments = new Regex(@"/\*.*?\*/", RegexOptions.Compiled);
         // IE6 <http://heideri.ch/jso/#80>
         private static readonly Regex CssExpression = new Regex(@"[eE\uFF25\uFF45][xX\uFF38\uFF58][pP\uFF30\uFF50][rR\u0280\uFF32\uFF52][eE\uFF25\uFF45][sS\uFF33\uFF53]{2}[iI\u026A\uFF29\uFF49][oO\uFF2F\uFF4F][nN\u0274\uFF2E\uFF4E]", RegexOptions.Compiled);
-        private static readonly Regex CssUrl = new Regex(@"[Uu][Rr\u0280][Ll\u029F]\s*\(\s*['""]?\s*([^'"")]+)", RegexOptions.Compiled);
+        private static readonly Regex CssUrl = new Regex(@"[Uu][Rr\u0280][Ll\u029F]\s*\(([^)]+)", RegexOptions.Compiled);
 
         /// <summary>
         /// Sanitizes the style.