Attempting a more reasonable fix for #372 and #373

cure53 · Oct 21, 2019 · 3a30371 · 3a30371
1 parent 967af07
commit 3a30371
Show file tree

Hide file tree

Showing 10 changed files with 11 additions and 17 deletions.
diff --git a/dist/purify.cjs.js b/dist/purify.cjs.js
diff --git a/dist/purify.cjs.js.map b/dist/purify.cjs.js.map
diff --git a/dist/purify.es.js b/dist/purify.es.js
diff --git a/dist/purify.es.js.map b/dist/purify.es.js.map
diff --git a/dist/purify.js b/dist/purify.js
diff --git a/dist/purify.js.map b/dist/purify.js.map
diff --git a/dist/purify.min.js b/dist/purify.min.js
diff --git a/dist/purify.min.js.map b/dist/purify.min.js.map
diff --git a/src/purify.js b/src/purify.js
@@ -486,9 +486,6 @@ function createDOMPurify(window = getGlobal()) {
       /* If FORCE_BODY isn't used, leading whitespace needs to be preserved manually */
       const matches = dirty.match(/^[\s]+/);
       leadingWhitespace = matches && matches[0];
-      if (leadingWhitespace) {
-        dirty = dirty.slice(leadingWhitespace.length);
-      }
     }
 
     /* Use DOMParser to workaround Firefox bug (see comment below) */

diff --git a/test/fixtures/expect.js b/test/fixtures/expect.js
@@ -1074,5 +1074,11 @@ module.exports = [
           "<svg><b><style><b></b></style></b></svg>",
           "<svg></svg><b><style><b title='</style><img>'&gt;</b>"
       ]
+  }, {
+      "title": "Tests against proper handling of leading whitespaces",
+      "payload": " ",
+      "expected": [
+          " "
+      ]
   }
 ];