chore(deps): update dependency fast-xml-parser to v5.3.6 [security]#1434
Merged
afwilcox merged 8 commits intorelease/2.17from Feb 18, 2026
Merged
Conversation
…github.com/bcgov/nr-compliance-enforcement into renovate/npm-fast-xml-parser-vulnerability
Contributor
Author
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
Collaborator
|
/patch_version |
|
afwilcox
approved these changes
Feb 18, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.



This PR contains the following updates:
5.3.2→5.3.6GitHub Vulnerability Alerts
CVE-2026-25128
Summary
A RangeError vulnerability exists in the numeric entity processing of fast-xml-parser when parsing XML with out-of-range entity code points (e.g.,
&#​9999999;or�). This causes the parser to throw an uncaught exception, crashing any application that processes untrusted XML input.Details
The vulnerability exists in
/src/xmlparser/OrderedObjParser.jsat lines 44-45:The
String.fromCodePoint()method throws aRangeErrorwhen the code point exceeds the valid Unicode range (0 to 0x10FFFF / 1114111). The regex patterns can capture values far exceeding this:[0-9]{1,7}matches up to 9,999,999[0-9a-fA-F]{1,6}matches up to 0xFFFFFF (16,777,215)The entity replacement in
replaceEntitiesValue()(line 452) has no try-catch:This causes the RangeError to propagate uncaught, crashing the parser and any application using it.
PoC
Setup
Create a directory with these files:
package.json
{ "dependencies": { "fast-xml-parser": "^5.3.3" } }server.js
Run
Result
Server crashes with:
Alternative Payloads
Impact
Denial of Service (DoS):* Any application using fast-xml-parser to process untrusted XML input will crash when encountering malformed numeric entities. This affects:
A single malicious request is sufficient to crash the entire Node.js process, causing service disruption until manual restart.
CVE-2026-26278
Summary
The XML parser can be forced to do an unlimited amount of entity expansion. With a very small XML input, it’s possible to make the parser spend seconds or even minutes processing a single request, effectively freezing the application.
Details
There is a check in
DocTypeReader.jsthat tries to prevent entity expansion attacks by rejecting entities that reference other entities (it looks for & inside entity values). This does stop classic “Billion Laughs” payloads.However, it doesn’t stop a much simpler variant.
If you define one large entity that contains only raw text (no & characters) and then reference it many times, the parser will happily expand it every time. There is no limit on how large the expanded result can become, or how many replacements are allowed.
The problem is in
replaceEntitiesValue()insideOrderedObjParser.js. It repeatedly runsval.replace()in a loop, without any checks on total output size or execution cost. As the entity grows or the number of references increases, parsing time explodes.Relevant code:
DocTypeReader.js(lines 28–33): entity registration only checks for &OrderedObjParser.js(lines 439–458): entity replacement loop with no limitsPoC
Impact
This is a straightforward denial-of-service issue.
Any service that parses user-supplied XML using the default configuration is vulnerable. Since Node.js runs on a single thread, the moment the parser starts expanding entities, the event loop is blocked. While this is happening, the server can’t handle any other requests.
In testing, a payload of only a few kilobytes was enough to make a simple HTTP server completely unresponsive for several minutes, with all other requests timing out.
Workaround
Avoid using DOCTYPE parsing by
processEntities: falseoption.Release Notes
NaturalIntelligence/fast-xml-parser (fast-xml-parser)
v5.3.6: Entity security and performanceCompare Source
maxEntitySize,maxExpansionDepth,maxTotalExpansions,maxExpandedLength,allowedTags,tagFilterFull Changelog: NaturalIntelligence/fast-xml-parser@v5.3.5...v5.3.6
v5.3.5Compare Source
v5.3.4: fix: handle HTML numeric and hex entities when out of rangeCompare Source
v5.3.3: bug fix and performance improvementsCompare Source
Configuration
📅 Schedule: Branch creation - "" in timezone America/Vancouver, Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.
Thanks for the PR!
Deployments, as required, will be available below:
Please create PRs in draft mode. Mark as ready to enable:
After merge, new images are deployed in: