Skip to content

fix: Patch 18 of 21 high dependency advisories - #99

Merged
ecv merged 1 commit into
mainfrom
fix/security-advisories
Oct 25, 2026
Merged

ecv merged 1 commit into
mainfrom
fix/security-advisories

Conversation

@ecv

@ecv ecv commented Oct 25, 2026

Copy link
Copy Markdown
Contributor

The Backstage portal has 21 open high advisories in its dependencies. They cover the MCP SDK, archive and FTP handling, HTTP proxy and redirect code, markdown rendering, and the Azure auth library.

This update fixes 18 of the 21 inside the portal's current major versions, except one Azure auth library step called out in the comment. The backend and app build, the type check passes, and the 17 unit tests pass.

Three stay open:

  • body-parser and multer: the patched releases are 2.5 days old and the repo's 3-day package age gate blocks them until 2026-10-25 11:39 UTC. A follow-up needs one yarn up -R after that.
  • node-forge: no patched release exists upstream yet (1.4.0 is the latest).

The open major-version updates #91 (react-router) and #97 (Node 24) are not needed for any of these advisories and are left alone.

Dependabot reports 21 high advisories on the Backstage portal. This
update takes every one that has a patched release old enough to pass
the 3-day package age gate. Remaining: body-parser, multer (released
2026-10-22, clear after 2026-10-25 11:39 UTC) and node-forge (no patched
release exists).

Key changes:
- Refresh the lockfile within current ranges for techdocs-node,
  grpc-js, the MCP SDK, follow-redirects, compression, launch-editor,
  markdown-it, http-cache-semantics, source-map-js and undici.
- Add resolutions where a parent pins an old version: @azure/msal-node
  ^7.1.0 (two majors above the parent's range, no fix in 5.x or 6.x),
  @graphql-tools/utils ^12.1.0, basic-ftp ^6.2.2, and undici under the
  module federation DTS plugin.
@ecv
ecv requested review from kevwilliams and scotwells October 25, 2026 04:30
@ecv

ecv commented Oct 25, 2026

Copy link
Copy Markdown
Contributor Author

Detail

Lockfile refresh within current ranges: @backstage/plugin-techdocs-node, @grpc/grpc-js, @modelcontextprotocol/sdk, follow-redirects, compression, launch-editor, markdown-it, http-cache-semantics, source-map-js, undici.

Resolutions added because a parent pins an old version: @azure/msal-node ^7.1.0 (the fix first ships in 7.1.0; @azure/identity requests ^6.0.0, so this is one major above its range and untested against a live Azure login), @graphql-tools/utils ^12.1.0 (pulled by mockttp and graphql-tools 8.x, test-time only), basic-ftp ^6.2.2 (parent requests ^5), and undici ^7.30.1 under @module-federation/dts-plugin.

@kevwilliams kevwilliams left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Verified no app-config or workspace package configures Azure or FTP, so the forced msal-node 7 and basic-ftp 6 majors sit on code paths production never runs.

@ecv
ecv merged commit 755a4b9 into main Oct 25, 2026
7 of 8 checks passed
@ecv
ecv deleted the fix/security-advisories branch October 25, 2026 04:58
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