Skip to content

fix(deps): override fast-uri and fast-xml-builder to patched versions#273

Merged
hdamker merged 1 commit into
camaraproject:mainfrom
hdamker:fix/transitive-vuln-overrides
May 11, 2026
Merged

fix(deps): override fast-uri and fast-xml-builder to patched versions#273
hdamker merged 1 commit into
camaraproject:mainfrom
hdamker:fix/transitive-vuln-overrides

Conversation

@hdamker
Copy link
Copy Markdown
Contributor

@hdamker hdamker commented May 9, 2026

What type of PR is this?

  • bug

What this PR does / why we need it:

Closes 4 open Dependabot alerts on transitive dependencies in validation/ by pinning fast-uri and fast-xml-builder to patched versions via package.json overrides.

Alert Package CVE Severity
#15 fast-uri CVE-2026-6322 (host confusion via percent-encoded @ / :) High
#14 fast-uri CVE-2026-6321 (path traversal via percent-encoded dot segments) High
#13 fast-xml-builder CVE-2026-44665 (attribute-quote bypass) High
#12 fast-xml-builder CVE-2026-44664 (comment-regex bypass) Medium

Both vulnerable packages are transitive — fast-uri under ajv, fast-xml-builder under fast-xml-parser. Parent ranges (ajv "fast-uri": "^3.0.1", fast-xml-parser "fast-xml-builder": "^1.1.5") already permit the patched versions but no upstream consumer release has shipped, so a package.json override is sufficient — no major bump of the direct deps needed. Same pattern as the existing lodash / postcss / protobufjs overrides added in tooling#265.

fast-xml-builder pinned to ~1.1.7 (not ^1.1.7) to stay on the patched 1.1.x line — 1.2.0 would otherwise introduce a brand-new xml-naming@0.1.0 transitive that the build has no use for.

Which issue(s) this PR fixes:

(no separate issue — Dependabot alerts auto-close on merge)

Special notes for reviewers:

Verification:

  • npm audit reports 0 vulnerabilities
  • 1011/1011 validation unit tests pass
  • Redocly bundle smoke test on ReleaseTest/sample-service.yaml succeeds
  • Same major versions of consumers (ajv 8.x, fast-xml-parser 5.7.x) — no behavior change expected

Should fold into the upcoming @v1-rc advance E2E alongside the other in-flight validation/ changes.

Changelog input

 release-note
 NONE

Additional documentation

docs
NONE

Closes 4 open Dependabot alerts in validation/ on transitive deps:
- fast-uri 3.1.0 -> 3.1.2 (GHSA-q3j6-qgpj-74h6, GHSA-v39h-62p7-jpjc)
- fast-xml-builder 1.1.5 -> 1.1.9 (GHSA-45c6-75p6-83cc, GHSA-5wm8-gmm8-39j9)

Parent ranges (ajv "fast-uri": "^3.0.1", fast-xml-parser "fast-xml-builder":
"^1.1.5") already permit the patched versions but no upstream bump has shipped,
so this pins via package.json overrides to match the existing lodash/postcss/
protobufjs pattern. fast-xml-builder pinned to ~1.1.7 to stay on the 1.1.x
line and avoid the new xml-naming@0.1.0 transitive that 1.2.0 introduces.

npm audit clean. 1011/1011 validation tests pass.
@hdamker hdamker requested review from Kevsy and rartych as code owners May 9, 2026 08:11
Copy link
Copy Markdown
Contributor

@Kevsy Kevsy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Contributor

@rartych rartych left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@hdamker hdamker merged commit 412039b into camaraproject:main May 11, 2026
1 check passed
@hdamker hdamker deleted the fix/transitive-vuln-overrides branch May 11, 2026 11:22
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.

3 participants