@instructure/platform-sanitize 0.3.17 → 0.4.1

package/dist/index.js

@@ -1,5 +1,5 @@
-import d from "dompurify";
-const m = /* @__PURE__ */ new Set([
+import f from "dompurify";
+const w = /* @__PURE__ */ new Set([
   // layout
   "display",
   "float",
@@ -241,7 +241,7 @@ const m = /* @__PURE__ */ new Set([
   "caret-color",
   "accent-color",
   "appearance"
-]), w = /* @__PURE__ */ new Set([
+]), h = /* @__PURE__ */ new Set([
   "src",
   "href",
   "action",
@@ -252,16 +252,7 @@ const m = /* @__PURE__ */ new Set([
   "cite",
   "longdesc",
   "xlink:href"
-]), h = [
-  "background",
-  "background-image",
-  "list-style",
-  "list-style-image",
-  "cursor",
-  // content: url(...) triggers an HTTP GET even on non-pseudo elements in some
-  // browsers; strip it as defense-in-depth against tracking-pixel exfiltration.
-  "content"
-], y = /url\s*\(\s*['"]?(?:[a-z][a-z0-9+\-.]*:|\/\/)/i, k = /* @__PURE__ */ new Set([
+]), y = ["content"], A = /url\s*\(\s*['"]?(?:[a-z][a-z0-9+\-.]*:|\/\/)/i, k = /* @__PURE__ */ new Set([
   "allow-downloads",
   "allow-forms",
   "allow-modals",
@@ -274,8 +265,22 @@ const m = /* @__PURE__ */ new Set([
   "allow-scripts",
   "allow-storage-access-by-user-activation",
   "allow-top-navigation-by-user-activation"
-]), f = {
-  ADD_TAGS: ["iframe"],
+]), p = {
+  // semantics and annotation are re-allowed because DOMPurify blocks all of
+  // mathMlDisallowed by default. semantics is a safe structural wrapper;
+  // annotation carries LaTeX source text (e.g. \frac{1}{2}); Canvas emits
+  // text-only content here, and any child elements would be hoisted out of
+  // the MathML context and sanitized as ordinary HTML.
+  //
+  // annotation-xml is re-added here and filtered by an afterSanitizeElements
+  // hook below: it is stripped when its encoding makes it an HTML integration
+  // point ("text/html" or "application/xhtml+xml" per the WHATWG spec), which
+  // is the mXSS namespace-confusion vector. Semantic encodings used by external
+  // tools (MathML-Content, MathML-Presentation from MathType/Wolfram Alpha,
+  // application/x-tex, …) are not integration points and are preserved, so this
+  // layer no longer strips the benign annotation-xml the backend and TinyMCE
+  // already allow.
+  ADD_TAGS: ["iframe", "semantics", "annotation", "annotation-xml"],
   ADD_ATTR: [
     "allowfullscreen",
     "allow",
@@ -319,17 +324,18 @@ const m = /* @__PURE__ */ new Set([
   // confusion attacks where fragments like <svg> could influence parse context.
   FORCE_BODY: !0
 };
-let a = null;
-function A() {
-  return a || (a = typeof d == "function" ? d(window) : d, a.addHook("afterSanitizeAttributes", (r) => {
-    if (!(r instanceof Element) || !r.hasAttribute("style")) return;
-    const t = r.style, e = [];
-    for (let n = 0; n < t.length; n++) {
-      const s = t.item(n);
-      m.has(s) || e.push(s);
+let s = null;
+function E() {
+  if (s) return s;
+  s = typeof f == "function" ? f(window) : f, s.addHook("afterSanitizeAttributes", (t) => {
+    if (!(t instanceof Element) || !t.hasAttribute("style")) return;
+    const e = t.style, r = [];
+    for (let i = 0; i < e.length; i++) {
+      const l = e.item(i);
+      w.has(l) || r.push(l);
     }
-    for (const n of e) t.removeProperty(n);
-    const o = /* @__PURE__ */ new Set([
+    for (const i of r) e.removeProperty(i);
+    const a = /* @__PURE__ */ new Set([
       "static",
       "relative",
       "absolute",
@@ -338,53 +344,59 @@ function A() {
       "unset",
       "revert",
       "revert-layer"
-    ]), l = t.getPropertyValue("position").trim().toLowerCase();
-    l && !o.has(l) && t.removeProperty("position");
-    const u = /* @__PURE__ */ new Set(["initial", "inherit", "unset", "revert", "revert-layer"]), i = t.getPropertyValue("opacity").trim().toLowerCase();
-    i && !u.has(i) && (i.endsWith("%") ? parseFloat(i) / 100 : parseFloat(i)) < 0.05 && t.removeProperty("opacity");
-    for (const n of h) {
-      const s = t.getPropertyValue(n);
-      s && y.test(s) && t.removeProperty(n);
+    ]), c = e.getPropertyValue("position").trim().toLowerCase();
+    c && !a.has(c) && e.removeProperty("position");
+    const d = /* @__PURE__ */ new Set(["initial", "inherit", "unset", "revert", "revert-layer"]), o = e.getPropertyValue("opacity").trim().toLowerCase();
+    o && !d.has(o) && (o.endsWith("%") ? parseFloat(o) / 100 : parseFloat(o)) < 0.05 && e.removeProperty("opacity");
+    for (const i of y) {
+      const l = e.getPropertyValue(i);
+      l && A.test(l) && e.removeProperty(i);
     }
-    t.length === 0 && r.removeAttribute("style");
-  }), a.addHook("uponSanitizeAttribute", (r, t) => {
-    if (!w.has(t.attrName)) return;
-    const e = t.attrValue;
-    /^\s*\/\//.test(e) ? (t.attrValue = e.trimStart().replace(/^\/\//, "https://"), t.keepAttr = !0) : /^\s*\\/.test(e) && (t.keepAttr = !1);
-  }), a.addHook("afterSanitizeAttributes", (r) => {
-    if (!(r instanceof Element) || !r.hasAttribute("srcset")) return;
-    const e = (r.getAttribute("srcset") ?? "").split(","), o = (i) => i.trim().split(/\s+/)[0];
-    if (e.some((i) => /^\s*\\/.test(o(i)))) {
-      r.removeAttribute("srcset");
+    e.length === 0 && t.removeAttribute("style");
+  }), s.addHook("uponSanitizeAttribute", (t, e) => {
+    if (!h.has(e.attrName)) return;
+    const r = e.attrValue;
+    /^\s*\/\//.test(r) ? (e.attrValue = r.trimStart().replace(/^\/\//, "https://"), e.keepAttr = !0) : /^\s*\\/.test(r) && (e.keepAttr = !1);
+  }), s.addHook("afterSanitizeAttributes", (t) => {
+    if (!(t instanceof Element) || !t.hasAttribute("srcset")) return;
+    const r = (t.getAttribute("srcset") ?? "").split(","), a = (o) => o.trim().split(/\s+/)[0];
+    if (r.some((o) => /^\s*\\/.test(a(o)))) {
+      t.removeAttribute("srcset");
       return;
     }
-    let l = !1;
-    const u = e.map((i) => {
-      const n = o(i);
-      if (!n.startsWith("//")) return i;
-      l = !0;
-      const s = i.indexOf(n);
-      return i.slice(0, s) + "https://" + n.slice(2) + i.slice(s + n.length);
+    let c = !1;
+    const d = r.map((o) => {
+      const i = a(o);
+      if (!i.startsWith("//")) return o;
+      c = !0;
+      const l = o.indexOf(i);
+      return o.slice(0, l) + "https://" + i.slice(2) + o.slice(l + i.length);
     });
-    l && r.setAttribute("srcset", u.join(","));
-  }), a.addHook("afterSanitizeAttributes", (r) => {
-    var o;
-    if (!(r instanceof Element) || r.tagName !== "A" && r.tagName !== "AREA" || ((o = r.getAttribute("target")) == null ? void 0 : o.toLowerCase()) !== "_blank") return;
-    const t = r.getAttribute("rel") ?? "", e = new Set(t.split(/\s+/).filter(Boolean));
-    e.add("noopener"), r.setAttribute("rel", [...e].join(" "));
-  }), a.addHook("afterSanitizeAttributes", (r) => {
-    if (!(r instanceof Element) || r.tagName !== "IFRAME" || !r.hasAttribute("sandbox")) return;
-    const e = (r.getAttribute("sandbox") ?? "").toLowerCase().split(/\s+/).filter(Boolean), o = e.filter((l) => k.has(l));
-    o.length !== e.length && r.setAttribute("sandbox", o.join(" "));
-  }), a);
+    c && t.setAttribute("srcset", d.join(","));
+  }), s.addHook("afterSanitizeAttributes", (t) => {
+    var a;
+    if (!(t instanceof Element) || t.tagName !== "A" && t.tagName !== "AREA" || ((a = t.getAttribute("target")) == null ? void 0 : a.toLowerCase()) !== "_blank") return;
+    const e = t.getAttribute("rel") ?? "", r = new Set(e.split(/\s+/).filter(Boolean));
+    r.add("noopener"), t.setAttribute("rel", [...r].join(" "));
+  }), s.addHook("afterSanitizeAttributes", (t) => {
+    if (!(t instanceof Element) || t.tagName !== "IFRAME" || !t.hasAttribute("sandbox")) return;
+    const r = (t.getAttribute("sandbox") ?? "").toLowerCase().split(/\s+/).filter(Boolean), a = r.filter((c) => k.has(c));
+    a.length !== r.length && t.setAttribute("sandbox", a.join(" "));
+  });
+  const n = /* @__PURE__ */ new Set(["text/html", "application/xhtml+xml"]);
+  return s.addHook("afterSanitizeElements", (t) => {
+    if (!(t instanceof Element) || t.tagName.toLowerCase() !== "annotation-xml") return;
+    const e = (t.getAttribute("encoding") ?? "").toLowerCase().trim();
+    n.has(e) && t.remove();
+  }), s;
 }
-function v(r, t) {
+function _(n, t) {
   if (typeof window > "u")
     throw new Error("sanitizeHtml requires a DOM environment (window is not defined)");
-  const e = t != null && t.allowFormAttributeNames ? { ...f, SANITIZE_DOM: !1 } : f;
-  return A().sanitize(r ?? "", e);
+  const e = t != null && t.allowFormAttributeNames ? { ...p, SANITIZE_DOM: !1 } : p;
+  return E().sanitize(n ?? "", e);
 }
-const p = /* @__PURE__ */ new Set(["http:", "https:", "mailto:", "tel:"]), c = "http://platform-sanitize.invalid/", g = /^\s*\/\//, b = (
+const m = /* @__PURE__ */ new Set(["http:", "https:", "mailto:", "tel:"]), u = "http://platform-sanitize.invalid/", g = /^\s*\/\//, b = (
   // oxlint-disable-next-line no-control-regex -- intentional security guard
   /^[\u0000-\u0020\u007F-\u00A0\u2000-\u200F\u2028\u2029\u202F\u205F\u2060\u3000\uFEFF]*(?:javascript|data|vbscript|file):/i
 ), S = {
@@ -400,49 +412,49 @@ const p = /* @__PURE__ */ new Set(["http:", "https:", "mailto:", "tel:"]), c = "
 `
   // java&NewLine;script:
 };
-function E(r) {
-  return r.replace(/&#(\d+);/g, (t, e) => {
-    const o = Number(e);
-    if (!Number.isFinite(o) || o < 0 || o > 1114111) return "";
+function x(n) {
+  return n.replace(/&#(\d+);/g, (t, e) => {
+    const r = Number(e);
+    if (!Number.isFinite(r) || r < 0 || r > 1114111) return "";
     try {
-      return String.fromCodePoint(o);
+      return String.fromCodePoint(r);
     } catch {
       return "";
     }
   }).replace(/&#x([\da-f]+);/gi, (t, e) => {
-    const o = parseInt(e, 16);
-    if (!Number.isFinite(o) || o < 0 || o > 1114111) return "";
+    const r = parseInt(e, 16);
+    if (!Number.isFinite(r) || r < 0 || r > 1114111) return "";
     try {
-      return String.fromCodePoint(o);
+      return String.fromCodePoint(r);
     } catch {
       return "";
     }
   }).replace(/&([A-Za-z][A-Za-z0-9]*);/g, (t, e) => S[e] ?? t);
 }
-function R(r) {
-  if (!r || !r.trim()) return "about:blank";
-  const t = r.replace(/\\/g, "/");
+function R(n) {
+  if (!n || !n.trim()) return "about:blank";
+  const t = n.replace(/\\/g, "/");
   if (g.test(t) || b.test(t)) return "about:blank";
   if (/&[#A-Za-z]/.test(t)) {
-    const e = E(t);
+    const e = x(t);
     if (g.test(e) || b.test(e))
       return "about:blank";
     try {
-      const o = new URL(e, c);
-      if (!o.href.startsWith(c) && !p.has(o.protocol))
+      const r = new URL(e, u);
+      if (!r.href.startsWith(u) && !m.has(r.protocol))
         return "about:blank";
     } catch {
       return "about:blank";
     }
   }
   try {
-    const e = new URL(t, c);
-    return !p.has(e.protocol) || (e.protocol === "http:" || e.protocol === "https:") && (e.username || e.password) ? "about:blank" : e.href.startsWith(c) ? r : t.replace(/[\x00-\x1F\u2028\u2029]/g, "").replace(/%250[9ad]/gi, "").replace(/%0[9ad]/gi, "");
+    const e = new URL(t, u);
+    return !m.has(e.protocol) || (e.protocol === "http:" || e.protocol === "https:") && (e.username || e.password) ? "about:blank" : e.href.startsWith(u) ? n : t.replace(/[\x00-\x1F\u2028\u2029]/g, "").replace(/%250[9ad]/gi, "").replace(/%0[9ad]/gi, "");
   } catch {
     return "about:blank";
   }
 }
 export {
-  v as sanitizeHtml,
+  _ as sanitizeHtml,
   R as sanitizeUrl
 };

package/dist/sanitizeHtml.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"sanitizeHtml.d.ts","sourceRoot":"","sources":["../src/sanitizeHtml.ts"],"names":[],"mappings":"AAgeA,wBAAgB,YAAY,CAC1B,IAAI,EAAE,MAAM,GAAG,IAAI,GAAG,SAAS,EAC/B,OAAO,CAAC,EAAE;IAAE,uBAAuB,CAAC,EAAE,OAAO,CAAA;CAAE,GAC9C,MAAM,CASR"}
+{"version":3,"file":"sanitizeHtml.d.ts","sourceRoot":"","sources":["../src/sanitizeHtml.ts"],"names":[],"mappings":"AA8fA,wBAAgB,YAAY,CAC1B,IAAI,EAAE,MAAM,GAAG,IAAI,GAAG,SAAS,EAC/B,OAAO,CAAC,EAAE;IAAE,uBAAuB,CAAC,EAAE,OAAO,CAAA;CAAE,GAC9C,MAAM,CASR"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@instructure/platform-sanitize",
-  "version": "0.3.17",
+  "version": "0.4.1",
   "type": "module",
   "main": "./dist/index.js",
   "module": "./dist/index.js",