Skip to content

fix: upgrade protobufjs to 6.11.6 & fix other issues when bumping related deps#2986

Draft
lochie wants to merge 16 commits into
mainfrom
lochiea/family-647-vanta-remediate-critical-vulnerabilities-identified-in
Draft

fix: upgrade protobufjs to 6.11.6 & fix other issues when bumping related deps#2986
lochie wants to merge 16 commits into
mainfrom
lochiea/family-647-vanta-remediate-critical-vulnerabilities-identified-in

Conversation

@lochie
Copy link
Copy Markdown
Contributor

@lochie lochie commented May 18, 2026

Resolves CVE-2026-41242 by forcing protobufjs to patched version 6.11.6. The vulnerability was present in protobufjs <7.5.5, and is backported to 6.x via the 6.11.6 release. This maintains API compatibility with existing consumers that depend on protobufjs ^6.x via @cowprotocol/cow-sdk."

Fixes Dependabot alert: https://github.com/aave/interface/security/dependabot/161

General Changes

  • Resolves "protobufjs": "^6.11.6"
  • Fixes broken clsx import in Link component
  • Updates next.js to v14 to fix build errors
  • Updates other vuln deps e.g qs
  • Pins react version types
  • Removes unused imports
  • Updates types for tanstack query

Reviewer Checklist

Please ensure you, as the reviewer(s), have gone through this checklist to ensure that the code changes are ready to ship safely and to help mitigate any downstream issues that may occur.

  • End-to-end tests are passing without any errors
  • Code changes do not significantly increase the application bundle size
  • If there are new 3rd-party packages, they do not introduce potential security threats
  • If there are new environment variables being added, they have been added to the .env.example file as well as the pertinant .github/actions/* files
  • There are no CI changes, or they have been approved by the DevOps and Engineering team(s)

@linear
Copy link
Copy Markdown

linear Bot commented May 18, 2026

FAMILY-647

@vercel
Copy link
Copy Markdown

vercel Bot commented May 18, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
interface Ready Ready Preview, Comment May 19, 2026 11:38am

Request Review

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 18, 2026

Dependency Review

The following issues were found:

  • ❌ 2 vulnerable package(s)
  • ⚠️ 26 packages with OpenSSF Scorecard issues.

View full job summary

forhau
forhau previously approved these changes May 18, 2026
lochie and others added 12 commits May 19, 2026 12:20
Resolves CVE-2026-41242 by forcing protobufjs to patched version 6.11.6.
The vulnerability was present in protobufjs <7.5.5, and is backported to
6.x via the 6.11.6 release. This maintains API compatibility with existing
consumers that depend on protobufjs ^6.x via @cowprotocol/cow-sdk."
resolve react@types from version * from other dependencies
This reverts commit 3172fdd.
This reverts commit 845abfd.
@sammdec sammdec force-pushed the lochiea/family-647-vanta-remediate-critical-vulnerabilities-identified-in branch from 9f2d7cc to 6d048f4 Compare May 19, 2026 11:29
@github-actions
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown

📦 Next.js Bundle Analysis for aave-ui

This analysis was generated by the Next.js Bundle Analysis action. 🤖

⚠️ Global Bundle Size Increased

Page Size (compressed)
global 1.2 MB (🟡 +46.78 KB)
Details

The global bundle is the javascript bundle that loads alongside every page. It is in its own category because its impact is much higher - an increase to its size means that every page on your website loads slower, and a decrease means every page loads faster.

Any third party scripts you have added directly to your app using the <script> tag are not accounted for in this analysis

If you want further insight into what is behind the changes, give @next/bundle-analyzer a try!

Eighteen Pages Changed Size

The following pages changed size from the code in this PR compared to its base branch:

Page Size (compressed) First Load
/ 67.4 KB (🟢 -673 B) 1.26 MB
/404 2.85 KB (🟢 -15 B) 1.2 MB
/500 3.18 KB (🟢 -18 B) 1.2 MB
/_error 1.98 KB (🟢 -53 B) 1.2 MB
/bridge 38.01 KB (🔴 +8.97 KB) 1.23 MB
/dashboard 56.94 KB (🟢 -639 B) 1.25 MB
/faucet 15.12 KB (🟢 -85 B) 1.21 MB
/governance 90.11 KB (🟡 +8.89 KB) 1.29 MB
/governance/ipfs-preview 101.64 KB (🟢 -405 B) 1.3 MB
/governance/v3/proposal 133.71 KB (🟡 +8.16 KB) 1.33 MB
/history 36.14 KB (🟡 +35 B) 1.23 MB
/markets 39.81 KB (🟢 -473 B) 1.24 MB
/reserve-overview 25.76 KB (🟢 -412 B) 1.22 MB
/safety-module 41.64 KB (🔴 +8.56 KB) 1.24 MB
/sentry-example 2.76 KB (🟢 -50 B) 1.2 MB
/sgho 80.26 KB (🟢 -837 B) 1.28 MB
/staking 33.48 KB (🟢 -218 B) 1.23 MB
/v3-migration 37.95 KB (🟡 +2 B) 1.23 MB
Details

Only the gzipped size is provided here based on an expert tip.

First Load is the size of the global bundle plus the bundle for the individual page. If a user were to show up to your website and land on a given page, the first load size represents the amount of javascript that user would need to download. If next/link is used, subsequent page loads would only need to download that page's bundle (the number in the "Size" column), since the global bundle has already been downloaded.

Any third party scripts you have added directly to your app using the <script> tag are not accounted for in this analysis

Next to the size is how much the size has increased or decreased compared with the base branch of this PR. If this percentage has increased by 20% or more, there will be a red status indicator applied, indicating that special attention should be given to this.

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