Skip to content

fix(extensions): bump yarn.lock packages for Dependabot CVEs - #4402

Open
kim-tsao wants to merge 4 commits into
redhat-developer:mainfrom
kim-tsao:chore/extensions-cve-bumps
Open

fix(extensions): bump yarn.lock packages for Dependabot CVEs#4402
kim-tsao wants to merge 4 commits into
redhat-developer:mainfrom
kim-tsao:chore/extensions-cve-bumps

Conversation

@kim-tsao

@kim-tsao kim-tsao commented Aug 19, 2026

Copy link
Copy Markdown
Member

Summary

  • yarn up -R on workspaces/extensions for open Dependabot alert packages, then yarn install and yarn dedupe.
  • Ancestor bump of leftover qs 6.14.2 via express / body-parser so the lockfile resolves only 6.15.3.

Fully fixed

package before after CVEs cleared
basic-ftp 5.0.5 5.3.1 CVE-2026-27699, GHSA-6v7q-wjvx-w8wg
brace-expansion 1.1.11, 2.1.2, 5.0.5 1.1.18, 2.1.4, 5.0.9 CVE-2026-13149
elliptic 6.6.0 6.6.1 GHSA-vjh7-7g9h-fjfh
fast-uri 3.0.3 3.1.5 CVE-2026-13676, CVE-2026-16221, CVE-2026-18446, CVE-2026-6321, CVE-2026-6322
follow-redirects 1.15.11 1.16.0 GHSA-r4q5-vmmm-2653
handlebars 4.7.8 4.7.9 CVE-2026-33916, CVE-2026-33937, CVE-2026-33938, CVE-2026-33940, CVE-2026-33941, GHSA-7rx3-28cr-v5wh
http-proxy-middleware 2.0.9, 3.0.5 2.0.10, 3.0.7 CVE-2026-55602
launch-editor 2.9.1 2.14.1 CVE-2026-53632
pbkdf2 3.1.2 3.1.6 CVE-2025-6545, CVE-2025-6547
picomatch 2.3.1, 4.0.4 2.3.2, 4.0.5 CVE-2026-33672
postcss 8.4.47 8.5.26 CVE-2026-41305, CVE-2026-45623, CVE-2026-69153, CVE-2026-73646
qs 6.14.2, 6.15.3 6.15.3 CVE-2026-8723
shell-quote 1.8.1 1.10.0 CVE-2026-13311, CVE-2026-9277
svgo 2.8.0 2.8.3 CVE-2026-29074, CVE-2026-73650
vm2 3.10.5 3.11.6 CVE-2026-43997, CVE-2026-43998, CVE-2026-43999, CVE-2026-44000, CVE-2026-44003, CVE-2026-44005, CVE-2026-44006, CVE-2026-44007, CVE-2026-47131, CVE-2026-47135, CVE-2026-47137, CVE-2026-47139, CVE-2026-47140, CVE-2026-47208, CVE-2026-47209, CVE-2026-47210, CVE-2026-47686, CVE-2026-47698, GHSA-2cm2-m3w5-gp2f, GHSA-m5w8-4gq2-6f8x
webpack-dev-server 5.2.2, 5.2.3 5.2.6 CVE-2026-14620, CVE-2026-14631, CVE-2026-6402, CVE-2026-9595
websocket-driver 0.7.4 0.7.5 CVE-2026-54466

Partial leftovers

package before after remaining
axios 1.13.6, 1.7.7 1.19.0, 1.7.7 1.7.7
dompurify 3.2.6, 3.4.8, 3.4.13 3.2.6, 3.4.8, 3.4.14 3.2.6, 3.4.8
ip-address 10.4.0, 9.0.5 10.5.0, 9.0.5 9.0.5
jsonpath-plus 10.3.0, 6.0.1, 7.1.0 10.4.0, 6.0.1, 7.1.0 6.0.1, 7.1.0
minimatch 3.1.2, 9.0.3, 10.2.5, … 3.1.2, 9.0.3, 10.2.6, … 3.1.2, 9.0.3
tar 6.2.1, 7.5.11 6.2.1, 7.5.22 6.2.1
undici 5.29.0, 7.24.7, 7.28.0 5.29.0, 7.24.7, 7.29.0 5.29.0, 7.24.7
@nestjs/common 10.4.6 10.4.6 needs 10.4.16
@nestjs/core 10.4.6 10.4.6 needs 11.1.18
adm-zip 0.5.10 0.5.10 needs 0.6.0
lodash 4.17.21, 4.17.23, 4.18.1 same 4.17.21
react-router 6.30.4 6.30.4 still 6.x; patched line is 7.18.0
tmp 0.0.33, 0.2.7 0.0.33, 0.2.7 0.0.33
uuid 10.0.0, 11.1.1, 3.4.0, 8.3.2, 9.0.1 same 3.x–10.x (11.1.1 already present)

Made with Cursor

Co-authored-by: Cursor <cursoragent@cursor.com>
@rhdh-qodo-merge

Copy link
Copy Markdown

PR Summary by Qodo

Remediate extension workspace dependency CVEs

🐞 Bug fix ⚙️ Configuration changes 🕐 20-40 Minutes

Grey Divider

AI Description

• Upgrades extension dependencies to patched versions for open Dependabot advisories.
• Regenerates and deduplicates the Yarn lockfile after recursive package updates.
• Preserves older versions where existing parent constraints prevent complete remediation.
Diagram

graph TD
  A["Dependabot Alerts"] --> B["Yarn Upgrade"] --> C["Extensions Lockfile"] --> D["Patched Packages"] --> F["Reduced CVEs"]
  C --> E["Updated Transitives"]
Loading
High-Level Assessment

The following are alternative approaches to this PR:

1. Upgrade parent dependencies
  • ➕ Can remove vulnerable legacy lines that remain in the lockfile
  • ➕ Avoids relying on transitive ranges to select patched releases
  • ➖ Expands scope into workspace manifest and application compatibility changes
  • ➖ May require framework migrations, especially NestJS and React Router
  • ➖ Requires broader build, test, and runtime validation
2. Add Yarn resolutions
  • ➕ Can force patched transitive versions without upgrading every parent package
  • ➕ May clear additional Dependabot alerts quickly
  • ➖ Can violate parent package compatibility expectations
  • ➖ Creates override maintenance burden
  • ➖ Cannot safely bridge every major-version remediation

Recommendation: Use the PR's lockfile refresh for compatible security patches because it minimizes application-level change while clearing many advisories. Address remaining vulnerable lines in focused follow-ups by upgrading their parent dependencies; reserve Yarn resolutions for verified, version-compatible exceptions.

Files changed (1) +155 / -139

Other (1) +155 / -139
yarn.lockRefresh vulnerable extension dependency resolutions +155/-139

Refresh vulnerable extension dependency resolutions

• Regenerates and deduplicates the extensions workspace lockfile, upgrading vulnerable packages including axios, basic-ftp, brace-expansion, elliptic, postcss, svgo, vm2, and websocket-driver. It also updates affected transitive dependency trees and checksums while retaining legacy versions constrained by existing parents.

workspaces/extensions/yarn.lock

@rhdh-qodo-merge

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Tip of the day
💡 Did you know, you can show, collapse, or hide each part of a finding: code, evidence, and all

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

@codecov

codecov Bot commented Aug 19, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.63%. Comparing base (2f7d169) to head (2fd9cc3).
⚠️ Report is 19 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4402   +/-   ##
=======================================
  Coverage   61.63%   61.63%           
=======================================
  Files        2541     2541           
  Lines      101955   101955           
  Branches    28576    28576           
=======================================
  Hits        62844    62844           
  Misses      38500    38500           
  Partials      611      611           
Flag Coverage Δ *Carryforward flag
adoption-insights 84.55% <ø> (ø) Carriedforward from d805d0c
ai-integrations 67.70% <ø> (ø) Carriedforward from d805d0c
app-defaults 48.37% <ø> (ø) Carriedforward from d805d0c
augment 46.67% <ø> (ø) Carriedforward from d805d0c
boost 79.79% <ø> (ø) Carriedforward from d805d0c
bulk-import 72.79% <ø> (ø) Carriedforward from d805d0c
cost-management 13.55% <ø> (ø) Carriedforward from d805d0c
dcm 72.09% <ø> (ø) Carriedforward from d805d0c
e2e-adoption-insights 60.00% <ø> (ø) Carriedforward from d805d0c
e2e-extensions 62.13% <ø> (ø) Carriedforward from d805d0c
e2e-global-header 49.45% <ø> (ø) Carriedforward from d805d0c
e2e-homepage 43.49% <ø> (ø) Carriedforward from d805d0c
e2e-intelligent-assistant 46.68% <ø> (ø) Carriedforward from d805d0c
e2e-orchestrator 49.51% <ø> (ø) Carriedforward from d805d0c
e2e-quickstart 55.21% <ø> (ø) Carriedforward from d805d0c
e2e-scorecard 50.21% <ø> (ø) Carriedforward from d805d0c
e2e-theme 16.36% <ø> (ø) Carriedforward from d805d0c
extensions 56.59% <ø> (ø)
global-floating-action-button 71.18% <ø> (ø) Carriedforward from d805d0c
global-header 66.50% <ø> (ø) Carriedforward from d805d0c
homepage 47.59% <ø> (ø) Carriedforward from d805d0c
install-dynamic-plugins 59.95% <ø> (ø) Carriedforward from d805d0c
intelligent-assistant 75.42% <ø> (ø) Carriedforward from d805d0c
konflux 91.98% <ø> (ø) Carriedforward from d805d0c
lightspeed 69.02% <ø> (ø) Carriedforward from d805d0c
mcp-integrations 83.40% <ø> (ø) Carriedforward from d805d0c
orchestrator 70.92% <ø> (ø) Carriedforward from d805d0c
quickstart 63.74% <ø> (ø) Carriedforward from d805d0c
sandbox 79.56% <ø> (ø) Carriedforward from d805d0c
scorecard 87.36% <ø> (ø) Carriedforward from d805d0c
theme 88.91% <ø> (ø) Carriedforward from d805d0c
translations 5.12% <ø> (ø) Carriedforward from d805d0c
x2a 79.20% <ø> (ø) Carriedforward from d805d0c

*This pull request uses carry forward flags. Click here to find out more.


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2f7d169...2fd9cc3. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@JessicaJHee JessicaJHee left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/lgtm

✅ Verified packages have been updated to the patched versions or beyond

  • Claimed full fixes in extensions match advisories (yarn why in workspaces/extensions/)

✅ Verified lockfile (workspaces/extensions/yarn.lock):

  • Minimal expected updates only (nanoid bump is required by postcss; ripemd160 / hash-base bumps are required by pbkdf2)
  • Single resolved version of each fully-fixed package after dedupe (picomatch, brace-expansion, and http-proxy-middleware keep patched major lines)
  • No dependency version downgrades
  • No unexpected @backstage/* bumps

✅ CI: required checks passing

Lift leftover qs 6.14.2 by bumping express/body-parser so the lockfile resolves only 6.15.3.

Co-authored-by: Cursor <cursoragent@cursor.com>
@openshift-ci openshift-ci Bot removed the lgtm label Aug 20, 2026
@openshift-ci

openshift-ci Bot commented Aug 20, 2026

Copy link
Copy Markdown

New changes are detected. LGTM label has been removed.

kim-tsao and others added 2 commits August 21, 2026 15:14
Co-authored-by: Cursor <cursoragent@cursor.com>
Keep CVE-2026-55602 fixed without jumping to ESM-only v4.

Co-authored-by: Cursor <cursoragent@cursor.com>
@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants