Skip to content

fix(deps): snyk follow-up console/frontend — react-router snooze + easy wins#2568

Open
eblairmckee wants to merge 1 commit into
masterfrom
chore/snyk-frontend-followup-2026-07-20
Open

fix(deps): snyk follow-up console/frontend — react-router snooze + easy wins#2568
eblairmckee wants to merge 1 commit into
masterfrom
chore/snyk-frontend-followup-2026-07-20

Conversation

@eblairmckee

Copy link
Copy Markdown
Contributor

What this does

Follow-up dependency-security PR for console/frontend, built off the latest master after the prior Snyk sweep merged. Two goals:

A) Snooze the non-reachable react-router advisory cluster

Console frontend is a client-only Module Federation SPA routed by TanStack Router. The vulnerable react-router SSR / RSC / data-router / single-fetch / prerender code paths never run in this bundle, so the whole cluster is non-reachable. Snoozed in frontend/.snyk with expiry 2026-10-20:

Package SNYK-JS ID
react-router SNYK-JS-REACTROUTER-17137394
react-router SNYK-JS-REACTROUTER-17138887
react-router SNYK-JS-REACTROUTER-17137545
react-router SNYK-JS-REACTROUTER-17138701
react-router SNYK-JS-REACTROUTER-17138883
react-router SNYK-JS-REACTROUTER-17342510
react-router SNYK-JS-REACTROUTER-14908531
@remix-run/router SNYK-JS-REMIXRUNROUTER-14908530

Root fix is tracked in #2567 (bump @redpanda-data/ui to peer a fixed react-router). react-router-dom has no direct Snyk advisories — all findings are attributed upstream to react-router / @remix-run/router.

B) Easy-win dependency fixes

Transitive build/test tooling only (none are imported in app source), bumped via overrides + resolutions (kept in sync). Verified locally: bun run type:check clean and all 928 unit tests pass.

Package Old New Advisory
adm-zip 0.5.10 0.6.0 GHSA-xcpc-8h2w-3j85
bn.js 4.12.2 / 5.2.2 5.2.5 GHSA-378v-28hj-76wf
brace-expansion 2.0.2 2.1.2 GHSA-f886-m6hf-6m8v
minimatch 5.1.6 / 9.0.5 9.0.9 GHSA-3ppc-4f35-3m26 et al.

Remaining, snoozed rather than bumped

  • elliptic (SNYK-JS-ELLIPTIC-14908844) — no upstream fix exists (Snyk: "There is no fix"; every published release incl. latest 6.6.1 is in range). Dev/build-only crypto polyfill via @rsbuild/plugin-node-polyfill › crypto-browserify › create-ecdh › elliptic; not reachable. Snoozed, expiry 2026-10-20.
  • uuid (SNYK-JS-UUID-16133035) — already snoozed in frontend/.snyk (non-reachable buffer-bounds API); left as-is.

Impact

bun audit: 24 -> 11 open findings. The 11 remaining are the react-router / @remix-run/router cluster + elliptic + uuid, all covered by .snyk snoozes above.

🤖 Generated with Claude Code

…sy wins

Follow-up to the merged Snyk sweep. Two changes:

A) Snooze the non-reachable react-router advisory cluster in frontend/.snyk.
   This is a client-only Module Federation SPA routed by TanStack Router; the
   vulnerable SSR/RSC/data-router/single-fetch/prerender code paths never run.
   Snoozed SNYK-JS IDs (expiry 2026-10-20):
   - react-router: 17137394, 17138887, 17137545, 17138701, 17138883,
     17342510, 14908531
   - @remix-run/router: 14908530 (REMIXRUNROUTER)
   Root fix tracked in #2567 (bump @redpanda-data/ui to
   peer a fixed react-router). react-router-dom has no direct Snyk advisories.

B) Easy-win transitive dependency bumps via overrides+resolutions (build/test
   tooling only, no app-source imports; type:check + 928 unit tests green):
   - adm-zip 0.5.10 -> 0.6.0        (GHSA-xcpc-8h2w-3j85)
   - bn.js 4.12.2/5.2.2 -> 5.2.5    (GHSA-378v-28hj-76wf)
   - brace-expansion 2.0.2 -> 2.1.2 (GHSA-f886-m6hf-6m8v)
   - minimatch 5.1.6/9.0.5 -> 9.0.9 (GHSA-3ppc-4f35-3m26 et al.)

   elliptic (SNYK-JS-ELLIPTIC-14908844) has no upstream fix — every published
   release incl. latest 6.6.1 is vulnerable — so it is snoozed rather than
   bumped; it is a dev/build-only crypto polyfill and not reachable.

Audit: 24 -> 11 open findings (remaining: react-router/@remix-run/router +
elliptic snoozed above, uuid already snoozed).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@eblairmckee eblairmckee added security Pull requests that address a security vulnerability team/ux dependencies snyk Snyk security sweep lang/ts TypeScript/JavaScript dismissals Snyk policy dismissals included labels Jul 20, 2026
@eblairmckee eblairmckee self-assigned this Jul 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies dismissals Snyk policy dismissals included lang/ts TypeScript/JavaScript security Pull requests that address a security vulnerability snyk Snyk security sweep team/ux

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants