@instructure/platform-sanitize 0.4.1 → 0.5.0

package/dist/index.js

@@ -280,7 +280,26 @@ const w = /* @__PURE__ */ new Set([
   // 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"],
+  // object/embed/param re-enable PDF/SVG/media embedding in course content
+  // (e.g. <object data="file.pdf" type="application/pdf">). Their Flash/ActiveX
+  // attributes (classid, codebase, pluginspage, wmode, allowscriptaccess) are
+  // intentionally NOT added — dead tech, and ADD_ATTR is global in DOMPurify so
+  // re-allowing them would also reopen that surface on iframe. DOMPurify's URI
+  // filter blocks javascript:/data: on data/src, matching Canvas's
+  // http/https/relative protocol rule for these tags.
+  //
+  // SECURITY ASSUMPTION (same one iframe already relies on): object/embed load
+  // their resource as a *document*, not an image — a same-origin SVG/HTML file
+  // (e.g. <object data="/files/123/download" type="image/svg+xml">) executes any
+  // <script> it contains in the embedding page's origin. The javascript:/data:
+  // filter does not help here because the URL is a benign same-origin path; the
+  // payload lives in the fetched file. This is NOT new attack surface: <iframe>
+  // is already allowed and has the identical capability (its srcdoc is stripped —
+  // see test). The defense for all three is that Canvas serves user-uploaded
+  // files from a separate sandboxed origin. object/embed also have no usable
+  // sandbox attribute, so the sandbox-token hook cannot constrain them — again,
+  // parity with an unsandboxed iframe, which is already permitted.
+  ADD_TAGS: ["iframe", "semantics", "annotation", "annotation-xml", "object", "embed", "param"],
   ADD_ATTR: [
     "allowfullscreen",
     "allow",
@@ -288,6 +307,12 @@ const w = /* @__PURE__ */ new Set([
     "sandbox",
     "data-media-id",
     "data-media-type",
+    // <object>'s resource URL attribute — not in DOMPurify's defaults. src/type/
+    // name/value/width/height (used by object/embed/param) already are. Added
+    // globally (DOMPurify has no per-tag attr scoping), which is harmless: `data`
+    // is in URL_ATTRS so it is protocol-filtered, and it is not a DOM-clobbering
+    // or URI-safe attribute name.
+    "data",
     // RCE produces target="_blank" for external links. Modern browsers treat
     // target="_blank" as implicit rel="noopener", and the backend allowlist
     // already includes target.
@@ -324,10 +349,10 @@ const w = /* @__PURE__ */ new Set([
   // confusion attacks where fragments like <svg> could influence parse context.
   FORCE_BODY: !0
 };
-let s = null;
+let n = null;
 function E() {
-  if (s) return s;
-  s = typeof f == "function" ? f(window) : f, s.addHook("afterSanitizeAttributes", (t) => {
+  if (n) return n;
+  n = typeof f == "function" ? f(window) : f, n.addHook("afterSanitizeAttributes", (t) => {
     if (!(t instanceof Element) || !t.hasAttribute("style")) return;
     const e = t.style, r = [];
     for (let i = 0; i < e.length; i++) {
@@ -335,7 +360,7 @@ function E() {
       w.has(l) || r.push(l);
     }
     for (const i of r) e.removeProperty(i);
-    const a = /* @__PURE__ */ new Set([
+    const s = /* @__PURE__ */ new Set([
       "static",
       "relative",
       "absolute",
@@ -345,7 +370,7 @@ function E() {
       "revert",
       "revert-layer"
     ]), c = e.getPropertyValue("position").trim().toLowerCase();
-    c && !a.has(c) && e.removeProperty("position");
+    c && !s.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) {
@@ -353,50 +378,54 @@ function E() {
       l && A.test(l) && e.removeProperty(i);
     }
     e.length === 0 && t.removeAttribute("style");
-  }), s.addHook("uponSanitizeAttribute", (t, e) => {
+  }), n.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) => {
+  }), n.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)))) {
+    const r = (t.getAttribute("srcset") ?? "").split(","), s = (o) => o.trim().split(/\s+/)[0];
+    if (r.some((o) => /^\s*\\/.test(s(o)))) {
       t.removeAttribute("srcset");
       return;
     }
     let c = !1;
     const d = r.map((o) => {
-      const i = a(o);
+      const i = s(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);
     });
     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;
+  }), n.addHook("afterSanitizeAttributes", (t) => {
+    var s;
+    if (!(t instanceof Element) || t.tagName !== "A" && t.tagName !== "AREA" || ((s = t.getAttribute("target")) == null ? void 0 : s.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) => {
+  }), n.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 r = (t.getAttribute("sandbox") ?? "").toLowerCase().split(/\s+/).filter(Boolean), s = r.filter((c) => k.has(c));
+    s.length !== r.length && t.setAttribute("sandbox", s.join(" "));
+  }), n.addHook("afterSanitizeAttributes", (t) => {
+    if (!(t instanceof Element) || t.tagName !== "PARAM") return;
+    const e = t.getAttribute("value");
+    e && /^\s*(?:javascript|vbscript):/i.test(e) && t.removeAttribute("value");
   });
-  const n = /* @__PURE__ */ new Set(["text/html", "application/xhtml+xml"]);
-  return s.addHook("afterSanitizeElements", (t) => {
+  const a = /* @__PURE__ */ new Set(["text/html", "application/xhtml+xml"]);
+  return n.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;
+    a.has(e) && t.remove();
+  }), n;
 }
-function _(n, t) {
+function R(a, t) {
   if (typeof window > "u")
     throw new Error("sanitizeHtml requires a DOM environment (window is not defined)");
   const e = t != null && t.allowFormAttributeNames ? { ...p, SANITIZE_DOM: !1 } : p;
-  return E().sanitize(n ?? "", e);
+  return E().sanitize(a ?? "", e);
 }
-const m = /* @__PURE__ */ new Set(["http:", "https:", "mailto:", "tel:"]), u = "http://platform-sanitize.invalid/", g = /^\s*\/\//, b = (
+const m = /* @__PURE__ */ new Set(["http:", "https:", "mailto:", "tel:"]), u = "http://platform-sanitize.invalid/", b = /^\s*\/\//, g = (
   // 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 = {
@@ -412,8 +441,8 @@ const m = /* @__PURE__ */ new Set(["http:", "https:", "mailto:", "tel:"]), u = "
 `
   // java&NewLine;script:
 };
-function x(n) {
-  return n.replace(/&#(\d+);/g, (t, e) => {
+function x(a) {
+  return a.replace(/&#(\d+);/g, (t, e) => {
     const r = Number(e);
     if (!Number.isFinite(r) || r < 0 || r > 1114111) return "";
     try {
@@ -431,13 +460,13 @@ function x(n) {
     }
   }).replace(/&([A-Za-z][A-Za-z0-9]*);/g, (t, e) => S[e] ?? t);
 }
-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";
+function _(a) {
+  if (!a || !a.trim()) return "about:blank";
+  const t = a.replace(/\\/g, "/");
+  if (b.test(t) || g.test(t)) return "about:blank";
   if (/&[#A-Za-z]/.test(t)) {
     const e = x(t);
-    if (g.test(e) || b.test(e))
+    if (b.test(e) || g.test(e))
       return "about:blank";
     try {
       const r = new URL(e, u);
@@ -449,12 +478,12 @@ function R(n) {
   }
   try {
     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, "");
+    return !m.has(e.protocol) || (e.protocol === "http:" || e.protocol === "https:") && (e.username || e.password) ? "about:blank" : e.href.startsWith(u) ? a : t.replace(/[\x00-\x1F\u2028\u2029]/g, "").replace(/%250[9ad]/gi, "").replace(/%0[9ad]/gi, "");
   } catch {
     return "about:blank";
   }
 }
 export {
-  _ as sanitizeHtml,
-  R as sanitizeUrl
+  R as sanitizeHtml,
+  _ as sanitizeUrl
 };

package/dist/sanitizeHtml.d.ts.map

@@ -1 +1 @@
-{"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"}
+{"version":3,"file":"sanitizeHtml.d.ts","sourceRoot":"","sources":["../src/sanitizeHtml.ts"],"names":[],"mappings":"AAoiBA,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.4.1",
+  "version": "0.5.0",
   "type": "module",
   "main": "./dist/index.js",
   "module": "./dist/index.js",