@instructure/platform-sanitize 0.3.15 → 0.4.0

package/dist/index.js

@@ -1,4 +1,4 @@
-import u from "dompurify";
+import d from "dompurify";
 const m = /* @__PURE__ */ new Set([
   // layout
   "display",
@@ -8,6 +8,7 @@ const m = /* @__PURE__ */ new Set([
   "overflow-x",
   "overflow-y",
   "visibility",
+  "opacity",
   "cursor",
   "direction",
   "user-select",
@@ -205,6 +206,10 @@ const m = /* @__PURE__ */ new Set([
   "animation-direction",
   "animation-fill-mode",
   "animation-play-state",
+  // transforms — no value filter needed; .user_content has overflow-y:hidden and overflow-x:auto
+  // (canvas-lms g/412928), which together contain transformed content within the container on
+  // both axes and prevent overlay of Canvas UI.
+  "transform",
   // columns
   "column-count",
   "column-width",
@@ -247,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([
+]), h = ["content"], y = /url\s*\(\s*['"]?(?:[a-z][a-z0-9+\-.]*:|\/\/)/i, A = /* @__PURE__ */ new Set([
   "allow-downloads",
   "allow-forms",
   "allow-modals",
@@ -285,13 +281,15 @@ const m = /* @__PURE__ */ new Set([
     "webkitallowfullscreen",
     "mozallowfullscreen",
     "scrolling",
+    // loading="lazy" is not in DOMPurify's default ALLOWED_ATTR and must be
+    // explicitly added. No security concern — it is a performance hint only.
+    "loading",
     // MathML 4 (W3C) annotation attributes for screen-reader accessibility.
     // MathCAT, JAWS, and NVDA use `intent` to know how to pronounce
     // expressions; `arg` labels sub-expressions referenced by intent.
     // These are plain string annotations — no URL-loading, no code execution.
     "intent",
-    "arg",
-    "loading"
+    "arg"
   ],
   // Rails UJS turns data-method/data-remote/etc. on clickable elements into
   // state-changing requests carrying the victim's CSRF token. Strip them so
@@ -313,15 +311,15 @@ const m = /* @__PURE__ */ new Set([
   FORCE_BODY: !0
 };
 let a = null;
-function A() {
-  return a || (a = typeof u == "function" ? u(window) : u, a.addHook("afterSanitizeAttributes", (e) => {
-    if (!(e instanceof Element) || !e.hasAttribute("style")) return;
-    const r = e.style, t = [];
-    for (let n = 0; n < r.length; n++) {
-      const i = r.item(n);
-      m.has(i) || t.push(i);
+function k() {
+  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);
     }
-    for (const n of t) r.removeProperty(n);
+    for (const n of e) t.removeProperty(n);
     const o = /* @__PURE__ */ new Set([
       "static",
       "relative",
@@ -331,54 +329,56 @@ function A() {
       "unset",
       "revert",
       "revert-layer"
-    ]), s = r.getPropertyValue("position").trim().toLowerCase();
-    s && !o.has(s) && r.removeProperty("position");
+    ]), 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 i = r.getPropertyValue(n);
-      i && y.test(i) && r.removeProperty(n);
+      const s = t.getPropertyValue(n);
+      s && y.test(s) && t.removeProperty(n);
     }
-    r.length === 0 && e.removeAttribute("style");
-  }), a.addHook("uponSanitizeAttribute", (e, r) => {
-    if (!w.has(r.attrName)) return;
-    const t = r.attrValue;
-    /^\s*\/\//.test(t) ? (r.attrValue = t.trimStart().replace(/^\/\//, "https://"), r.keepAttr = !0) : /^\s*\\/.test(t) && (r.keepAttr = !1);
-  }), a.addHook("afterSanitizeAttributes", (e) => {
-    if (!(e instanceof Element) || !e.hasAttribute("srcset")) return;
-    const t = (e.getAttribute("srcset") ?? "").split(","), o = (i) => i.trim().split(/\s+/)[0];
-    if (t.some((i) => /^\s*\\/.test(o(i)))) {
-      e.removeAttribute("srcset");
+    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");
       return;
     }
-    let s = !1;
-    const n = t.map((i) => {
-      const l = o(i);
-      if (!l.startsWith("//")) return i;
-      s = !0;
-      const d = i.indexOf(l);
-      return i.slice(0, d) + "https://" + l.slice(2) + i.slice(d + l.length);
+    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);
     });
-    s && e.setAttribute("srcset", n.join(","));
-  }), a.addHook("afterSanitizeAttributes", (e) => {
+    l && r.setAttribute("srcset", u.join(","));
+  }), a.addHook("afterSanitizeAttributes", (r) => {
     var o;
-    if (!(e instanceof Element) || e.tagName !== "A" && e.tagName !== "AREA" || ((o = e.getAttribute("target")) == null ? void 0 : o.toLowerCase()) !== "_blank") return;
-    const r = e.getAttribute("rel") ?? "", t = new Set(r.split(/\s+/).filter(Boolean));
-    t.add("noopener"), e.setAttribute("rel", [...t].join(" "));
-  }), a.addHook("afterSanitizeAttributes", (e) => {
-    if (!(e instanceof Element) || e.tagName !== "IFRAME" || !e.hasAttribute("sandbox")) return;
-    const t = (e.getAttribute("sandbox") ?? "").toLowerCase().split(/\s+/).filter(Boolean), o = t.filter((s) => k.has(s));
-    o.length !== t.length && e.setAttribute("sandbox", o.join(" "));
+    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) => A.has(l));
+    o.length !== e.length && r.setAttribute("sandbox", o.join(" "));
   }), a);
 }
-function v(e, r) {
+function v(r, t) {
   if (typeof window > "u")
     throw new Error("sanitizeHtml requires a DOM environment (window is not defined)");
-  const t = r != null && r.allowFormAttributeNames ? { ...f, SANITIZE_DOM: !1 } : f;
-  return A().sanitize(e ?? "", t);
+  const e = t != null && t.allowFormAttributeNames ? { ...f, SANITIZE_DOM: !1 } : f;
+  return k().sanitize(r ?? "", e);
 }
-const p = /* @__PURE__ */ new Set(["http:", "https:", "mailto:", "tel:"]), c = "http://platform-sanitize.invalid/", g = /^\s*\/\//, b = (
+const p = /* @__PURE__ */ new Set(["http:", "https:", "mailto:", "tel:"]), c = "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
-), x = {
+), S = {
   colon: ":",
   // javascript&colon;alert(1) → javascript:alert(1)
   sol: "/",
@@ -391,35 +391,35 @@ const p = /* @__PURE__ */ new Set(["http:", "https:", "mailto:", "tel:"]), c = "
 `
   // java&NewLine;script:
 };
-function E(e) {
-  return e.replace(/&#(\d+);/g, (r, t) => {
-    const o = Number(t);
+function E(r) {
+  return r.replace(/&#(\d+);/g, (t, e) => {
+    const o = Number(e);
     if (!Number.isFinite(o) || o < 0 || o > 1114111) return "";
     try {
       return String.fromCodePoint(o);
     } catch {
       return "";
     }
-  }).replace(/&#x([\da-f]+);/gi, (r, t) => {
-    const o = parseInt(t, 16);
+  }).replace(/&#x([\da-f]+);/gi, (t, e) => {
+    const o = parseInt(e, 16);
     if (!Number.isFinite(o) || o < 0 || o > 1114111) return "";
     try {
       return String.fromCodePoint(o);
     } catch {
       return "";
     }
-  }).replace(/&([A-Za-z][A-Za-z0-9]*);/g, (r, t) => x[t] ?? r);
+  }).replace(/&([A-Za-z][A-Za-z0-9]*);/g, (t, e) => S[e] ?? t);
 }
-function R(e) {
-  if (!e || !e.trim()) return "about:blank";
-  const r = e.replace(/\\/g, "/");
-  if (g.test(r) || b.test(r)) return "about:blank";
-  if (/&[#A-Za-z]/.test(r)) {
-    const t = E(r);
-    if (g.test(t) || b.test(t))
+function R(r) {
+  if (!r || !r.trim()) return "about:blank";
+  const t = r.replace(/\\/g, "/");
+  if (b.test(t) || g.test(t)) return "about:blank";
+  if (/&[#A-Za-z]/.test(t)) {
+    const e = E(t);
+    if (b.test(e) || g.test(e))
       return "about:blank";
     try {
-      const o = new URL(t, c);
+      const o = new URL(e, c);
       if (!o.href.startsWith(c) && !p.has(o.protocol))
         return "about:blank";
     } catch {
@@ -427,8 +427,8 @@ function R(e) {
     }
   }
   try {
-    const t = new URL(r, c);
-    return !p.has(t.protocol) || (t.protocol === "http:" || t.protocol === "https:") && (t.username || t.password) ? "about:blank" : t.href.startsWith(c) ? e : r.replace(/[\x00-\x1F\u2028\u2029]/g, "").replace(/%250[9ad]/gi, "").replace(/%0[9ad]/gi, "");
+    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, "");
   } catch {
     return "about:blank";
   }

package/dist/sanitizeHtml.d.ts.map

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