[Snyk] Security upgrade @xmldom/xmldom from 0.8.15 to 0.9.12 - #4637
[Snyk] Security upgrade @xmldom/xmldom from 0.8.15 to 0.9.12#4637chelojimenez wants to merge 1 commit into
Conversation
The following vulnerabilities are fixed with an upgrade: - https://snyk.io/vuln/SNYK-JS-XMLDOMXMLDOM-19498551
|
The upgrade of Key Breaking Changes:
Recommendation: Source: Release notes
|
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_d05e0a35-6cd4-4271-a322-f348f426d11f) |
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
There was a problem hiding this comment.
2 issues found across 1 file
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="mcpjam-inspector/package.json">
<violation number="1" location="mcpjam-inspector/package.json:151">
P1: Clean workspace installs now fail because the root lockfile still resolves `@xmldom/xmldom` to 0.8.13 and has no 0.9.12 entry. Regenerate and commit the root `package-lock.json` with this dependency bump.</violation>
<violation number="2" location="mcpjam-inspector/package.json:151">
P1: This direct bump does not remove the vulnerable 0.8.x parser from the workspace: `sdk` and `xml-crypto` still require incompatible 0.8.x ranges, forcing npm to install an older copy as well. Upgrade or override every consumer of `@xmldom/xmldom` (including `xml-crypto`), then regenerate the lockfile so the vulnerable version is absent.</violation>
</file>
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
| "@workos-inc/authkit-react": "^0.12.0", | ||
| "@workos-inc/node": "^10.2.0", | ||
| "@xmldom/xmldom": "^0.8.13", | ||
| "@xmldom/xmldom": "^0.9.12", |
There was a problem hiding this comment.
P1: Clean workspace installs now fail because the root lockfile still resolves @xmldom/xmldom to 0.8.13 and has no 0.9.12 entry. Regenerate and commit the root package-lock.json with this dependency bump.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At mcpjam-inspector/package.json, line 151:
<comment>Clean workspace installs now fail because the root lockfile still resolves `@xmldom/xmldom` to 0.8.13 and has no 0.9.12 entry. Regenerate and commit the root `package-lock.json` with this dependency bump.</comment>
<file context>
@@ -148,7 +148,7 @@
"@workos-inc/authkit-react": "^0.12.0",
"@workos-inc/node": "^10.2.0",
- "@xmldom/xmldom": "^0.8.13",
+ "@xmldom/xmldom": "^0.9.12",
"@xterm/addon-clipboard": "^0.2.0",
"@xterm/addon-fit": "^0.11.0",
</file context>
| "@workos-inc/authkit-react": "^0.12.0", | ||
| "@workos-inc/node": "^10.2.0", | ||
| "@xmldom/xmldom": "^0.8.13", | ||
| "@xmldom/xmldom": "^0.9.12", |
There was a problem hiding this comment.
P1: This direct bump does not remove the vulnerable 0.8.x parser from the workspace: sdk and xml-crypto still require incompatible 0.8.x ranges, forcing npm to install an older copy as well. Upgrade or override every consumer of @xmldom/xmldom (including xml-crypto), then regenerate the lockfile so the vulnerable version is absent.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At mcpjam-inspector/package.json, line 151:
<comment>This direct bump does not remove the vulnerable 0.8.x parser from the workspace: `sdk` and `xml-crypto` still require incompatible 0.8.x ranges, forcing npm to install an older copy as well. Upgrade or override every consumer of `@xmldom/xmldom` (including `xml-crypto`), then regenerate the lockfile so the vulnerable version is absent.</comment>
<file context>
@@ -148,7 +148,7 @@
"@workos-inc/authkit-react": "^0.12.0",
"@workos-inc/node": "^10.2.0",
- "@xmldom/xmldom": "^0.8.13",
+ "@xmldom/xmldom": "^0.9.12",
"@xterm/addon-clipboard": "^0.2.0",
"@xterm/addon-fit": "^0.11.0",
</file context>
Snyk has created this PR to fix 1 vulnerabilities in the npm dependencies of this project.
Snyk changed the following file(s):
mcpjam-inspector/package.jsonVulnerabilities that will be fixed with an upgrade:
SNYK-JS-XMLDOMXMLDOM-19498551
Breaking Change Risk
Important
Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic
Learn how to fix vulnerabilities with free interactive lessons:
🦉 Regular Expression Denial of Service (ReDoS)
Note
Medium Risk
Single dependency bump across a minor major line (0.8→0.9) for a parser used on the server; low code churn but possible parsing/API behavior changes where XML is handled.
Overview
Bumps
@xmldom/xmldominmcpjam-inspector/package.jsonfrom the 0.8.x range to^0.9.12, a Snyk-driven security fix for a high-severity ReDoS issue (SNYK-JS-XMLDOMXMLDOM-19498551).There are no application code changes—only the declared dependency version. The package remains part of the server bundle externals (alongside
xml-crypto), so runtime XML parsing behavior may change with the new release; smoke-testing XML/SAML or crypto-related flows is worthwhile after install.Reviewed by Cursor Bugbot for commit 230276d. Bugbot is set up for automated code reviews on this repo. Configure here.
Summary by cubic
Updates
@xmldom/xmldomfrom0.8.15to0.9.12to address a high-severity Regular Expression Denial of Service vulnerability.Written for commit 230276d. Summary will update on new commits.