Skip to content

Replace xmldom with jsdom#68

Merged
azhong-git merged 2 commits intomainfrom
replace-xmldom-with-jsdom
Apr 7, 2026
Merged

Replace xmldom with jsdom#68
azhong-git merged 2 commits intomainfrom
replace-xmldom-with-jsdom

Conversation

@azhong-git
Copy link
Copy Markdown
Contributor

Replace xmldom which has critical vulnerability.

xmldom * Severity: critical xmldom allows multiple root nodes in a DOM - https://github.com/advisories/GHSA-crh6-fp67-6883 Misinterpretation of malicious XML input - https://github.com/advisories/GHSA-5fg8-2547-mr8q xmldom: XML injection via unsafe CDATA serialization allows attacker-controlled markup insertion - https://github.com/advisories/GHSA-wh4c-j3r5-mjhp

The jsdom 24.x pin is because jsdom 25+ pulls in ESM-only transitive dependencies (@csstools/*, parse5 v8) that break Jest's CommonJS module resolution. When the project eventually moves to an ESM-native test runner or Jest adds better ESM support, you can upgrade to jsdom 27.x.

@azhong-git azhong-git temporarily deployed to integration-tests April 7, 2026 03:10 — with GitHub Actions Inactive
@azhong-git azhong-git merged commit 7158229 into main Apr 7, 2026
5 checks passed
@azhong-git azhong-git deleted the replace-xmldom-with-jsdom branch April 7, 2026 03:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant