Skip to content

security: clear all high-severity Dependabot alerts (ws, fast-uri, protobufjs, fast-xml-builder)#975

Merged
willwashburn merged 3 commits into
mainfrom
claude/compassionate-meitner-knD6s
May 24, 2026
Merged

security: clear all high-severity Dependabot alerts (ws, fast-uri, protobufjs, fast-xml-builder)#975
willwashburn merged 3 commits into
mainfrom
claude/compassionate-meitner-knD6s

Conversation

@willwashburn
Copy link
Copy Markdown
Member

@willwashburn willwashburn commented May 24, 2026

Summary

Lockfile-only bumps that clear all four high-severity Dependabot alerts plus one moderate, with zero impact on how relay works. Every entry below is a pure transitive (or already-in-range direct) dep update — no package.json, no source, no API changes.

ws 8.20.0 → 8.21.0 (moderate)

Existing constraints (^8.18.3 in root + packages/sdk, ^8.0.0 in packages/openclaw) already permit 8.21.0.

fast-uri 3.1.0 → 3.1.2 (high)

protobufjs 7.5.5 → 7.6.1 (high)

fast-xml-builder 1.1.5 → 1.2.0 (high)

What changed

  • package-lock.json only — no package.json files modified, no source code touched.
  • Generated via npm update <pkg> --package-lock-only.
  • All bumps are semver-compatible (patch or minor) on transitive or already-allowed direct deps.

Impact on relay

None. None of these packages are imported directly anywhere in relay's source — protobufjs, fast-uri, and fast-xml-builder are pure transitive deps, and ws stays within the existing ^8.x constraint.

Verification

$ npm audit            # before this PR: 21 vulns (3 high, 18 moderate)
$ npm audit            # after  this PR: 16 vulns (0 high, 16 moderate)

All five fixed packages (ws, fast-uri, protobufjs, @protobufjs/utf8, fast-xml-builder) are no longer listed.

Test plan

  • CI passes (typecheck + builds + tests)
  • npm audit reports no high-severity vulnerabilities

Resolves GHSA-58qx-3vcg-4xpx (CVE on ws >=8.0.0 <8.20.1). Lockfile-only
change; declared `^8.18.3` already accepts the patched version.
@willwashburn willwashburn requested a review from khaliqgant as a code owner May 24, 2026 04:39
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 24, 2026

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 7707e2b5-4500-408c-8a32-54eb05f9a5af

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/compassionate-meitner-knD6s

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

Re-trigger cubic

…sion

Resolves GHSA-q3j6-qgpj-74h6 (path traversal via percent-encoded dot
segments) and GHSA-v39h-62p7-jpjc (host confusion via percent-encoded
authority delimiters). Lockfile-only change; fast-uri is a pure
transitive dependency.
@willwashburn willwashburn changed the title security: bump ws to 8.21.0 (GHSA-58qx-3vcg-4xpx) security: clear ws and fast-uri Dependabot alerts May 24, 2026
… alerts

- protobufjs 7.5.5 → 7.6.1 resolves GHSA-66ff-xgx4-vchm (code injection
  via bytes-field defaults), GHSA-2pr8-phx7-x9h3, GHSA-fx83-v9x8-x52w,
  GHSA-75px-5xx7-5xc7, GHSA-jvwf-75h9-cwgg, GHSA-685m-2w69-288q,
  GHSA-q6x5-8v7m-xcrf, and GHSA-jggg-4jg4-v7c6.
- fast-xml-builder 1.1.5 → 1.2.0 resolves GHSA-5wm8-gmm8-39j9
  (attribute-value quote bypass) and GHSA-45c6-75p6-83cc.

Both are pure transitive dependencies; lockfile-only minor bumps.
@willwashburn willwashburn changed the title security: clear ws and fast-uri Dependabot alerts security: clear all high-severity Dependabot alerts (ws, fast-uri, protobufjs, fast-xml-builder) May 24, 2026
@willwashburn willwashburn merged commit ae28c3c into main May 24, 2026
39 checks passed
@willwashburn willwashburn deleted the claude/compassionate-meitner-knD6s branch May 24, 2026 14:02
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.

2 participants