Merge pull request #329 from rtfpessoa/fix-escaping-long-lines

fix: Enforce escape for lines that are not highlighted
rtfpessoa · Jul 26, 2020 · 4162188 · 4162188
2 parents c7c56bc + a61c7cf
commit 4162188
Show file tree

Hide file tree

Showing 3 changed files with 311 additions and 2 deletions.
diff --git a/src/__tests__/diff2html-tests.ts b/src/__tests__/diff2html-tests.ts
diff --git a/src/__tests__/diffs/bad-escaping.diff b/src/__tests__/diffs/bad-escaping.diff
diff --git a/src/render-utils.ts b/src/render-utils.ts
@@ -229,11 +229,11 @@ export function diffHighlight(
     return {
       oldLine: {
         prefix: line1.prefix,
-        content: line1.content,
+        content: escapeForHtml(line1.content),
       },
       newLine: {
         prefix: line2.prefix,
-        content: line2.content,
+        content: escapeForHtml(line2.content),
       },
     };
   }