-
Notifications
You must be signed in to change notification settings - Fork 5.5k
chore: Upgrade fast-xml-parser to 5.3.4 (GHSA-37qj-frw5-hhjh)
#39683
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
- Add resolution in package.json for fast-xml-parser ^5.3.4 - Add fast-xml-parser to npmPreapprovedPackages for minimal age gate; remove once satisfied
|
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
fast-xml-parser to 5.3.4 (GHSA-37qj-frw5-hhjh)
Builds ready [e953db8]
UI Startup Metrics (1293 ± 106 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs
|
|
@metamaskbot update-policies |
|
Policies updated. 🧠 Learn how: https://lavamoat.github.io/guides/policy-diff/#what-to-look-for-when-reviewing-a-policy-diff 👀 lavamoat/browserify/beta/policy.json changes differ from main/policy.json policy changes |
✨ Files requiring CODEOWNER review ✨📜 @MetaMask/policy-reviewers (8 files, +4 -44)
Tip Follow the policy review process outlined in the LavaMoat Policy Review Process doc before expecting an approval from Policy Reviewers. |
Gudahtt
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
mcmire
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Builds ready [57261b8]
UI Startup Metrics (1320 ± 107 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs
|
Description
Reason: fast-xml-parser 4.x (pulled in by @metamask/snaps-utils) is vulnerable to a RangeError DoS (GHSA-37qj-frw5-hhjh).
fast-xml-parserChangelogThe only breaking change in v5 is the ESM support
Solution: Add a resolution to fast-xml-parser ^5.3.4 and add it to
npmPreapprovedPackagesso the safe version can be installed despite the minimal age gate.Changelog
CHANGELOG entry: null
Related issues
Fixes:
Manual testing steps
Screenshots/Recordings
N/A (dependency upgrade).
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist
Note
Medium Risk
Dependency upgrade plus LavaMoat policy updates could cause runtime/build regressions if
@metamask/snaps-utilsexpects v4 behavior or if the new globals aren’t correctly captured.Overview
Upgrades
fast-xml-parserto^5.3.4viapackage.jsonresolutions (and updatesyarn.lock, includingstrnumto^2.1.0) to address GHSA-37qj-frw5-hhjh.Refreshes LavaMoat policies: removes the
@metamask/snaps-utils>fast-xml-parserallowlist entry from Browserify policies, and updates Webpack MV2 policies to allow the newfast-xml-parseraccess pattern (exports.isExist) instead of the oldentityName/valglobals.Written by Cursor Bugbot for commit 57261b8. This will update automatically on new commits. Configure here.