Skip to content

chore: resolve open dependabot security alerts#421

Draft
jonathannorris wants to merge 9 commits into
mainfrom
chore/dependabot-alerts
Draft

chore: resolve open dependabot security alerts#421
jonathannorris wants to merge 9 commits into
mainfrom
chore/dependabot-alerts

Conversation

@jonathannorris

@jonathannorris jonathannorris commented May 8, 2026

Copy link
Copy Markdown
Member

Summary

Addresses all open Dependabot security alerts by bumping direct deps where possible, adding overrides only where a major-version jump is required.

Dependabot Alerts Resolved

Alert Package Severity Fix
#128, #171 serialize-javascript high/medium Override to ^7.0.5 (required: copy-webpack-plugin / css-minimizer-webpack-plugin pin ^6.x)
#159, #160 fast-uri high Resolved naturally via OTel dep updates (lockfile: 3.1.2)
#161 @babel/plugin-transform-modules-systemjs high Resolved naturally via OTel dep updates (lockfile: 7.29.4)
#164 @opentelemetry/auto-instrumentations-node high Direct dep bumped to ^0.76.0
#165 @opentelemetry/sdk-node high Direct dep bumped to ^0.218.0
#168 protobufjs medium Resolved naturally; removed stale override (lockfile: 7.6.1)
#169 uuid medium Override to ^11.1.1 (no patch available in v8/v9/v10 line)
#170 @tootallnate/once low Resolved naturally; http-proxy-agent's ^2 range allows 2.0.1
#172 qs medium Override bumped to ^6.15.2
#173 tmp high Resolved naturally; NX's ~0.2.1 range allows 0.2.7
#174, #175, #176, #177 axios high/medium/low Override bumped to ^1.16.0
#69, #70, #166 webpack-dev-server medium Override to ^5.2.4 (required: @nx/webpack@17 pins ^4.x)

Notes

  • npm install requires --legacy-peer-deps due to a pre-existing @nestjs/schematics / prettier peer dependency conflict unrelated to this change.

Signed-off-by: Jonathan Norris <jonathan.norris@dynatrace.com>

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request updates several dependencies, including upgrading serialize-javascript to version 7.0.5 and @tootallnate/once to version 3.0.1, while adding minimatch and removing randombytes. Feedback highlights that the serialize-javascript upgrade introduces a breaking requirement for Node.js 20.0.0, which may impact environments on older LTS versions. Additionally, it is recommended to use caret ranges instead of the >= operator in the dependency overrides to ensure stability and prevent unintended major version upgrades.

Comment thread package-lock.json
Comment thread package.json Outdated
- fast-uri 3.1.0 -> 3.1.2 (high, alerts #159 #160)
- @babel/plugin-transform-modules-systemjs 7.29.0 -> 7.29.4 (high, alert #161)

Signed-off-by: Jonathan Norris <jonathan.norris@dynatrace.com>
Signed-off-by: Jonathan Norris <jonathan.norris@dynatrace.com>
@jonathannorris jonathannorris marked this pull request as draft May 11, 2026 14:11
@jonathannorris jonathannorris marked this pull request as ready for review May 12, 2026 19:10

Copilot AI 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.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds npm overrides to mitigate several Dependabot security alerts by forcing patched versions of vulnerable transitive dependencies.

Changes:

  • Added overrides for serialize-javascript, @tootallnate/once, fast-uri, and @babel/plugin-transform-modules-systemjs.
  • Kept existing test-exclude override (with minimatch) and extended the override list.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread package.json Outdated
Comment thread package.json Outdated
- Bump @opentelemetry/sdk-node to ^0.217.0 (alert #165, high)
- Bump @opentelemetry/auto-instrumentations-node to ^0.75.0 (alert #164, high)
- Bump related @opentelemetry/* packages to OTel 2.x for compatibility
- Update scripts/tracing.js to use Resource API from OTel 2.x
- Add webpack-dev-server ^5.2.4 override (alerts #69, #70, #166, medium)

Signed-off-by: Jonathan Norris <jonathan.norris@dynatrace.com>
Signed-off-by: Jonathan Norris <jonathan.norris@dynatrace.com>
- Remove fast-uri override (ajv ^3.0.1 range already covers patched 3.1.2)
- Remove @babel/plugin-transform-modules-systemjs override (@babel/preset-env ^7.29.0 already covers patched 7.29.4)
- Downgrade @tootallnate/once override from ^3.0.1 to ^2.0.1 (http-proxy-agent requires major-2; patched 2.0.1 is in range)
- Remove protobufjs override; update @opentelemetry packages to 0.218.0 which drops the protobufjs dependency entirely

Signed-off-by: Jonathan Norris <jonathan.norris@dynatrace.com>
Signed-off-by: Jonathan Norris <jonathan.norris@dynatrace.com>
Signed-off-by: Jonathan Norris <jonathan.norris@dynatrace.com>
Signed-off-by: Jonathan Norris <jonathan.norris@dynatrace.com>
@jonathannorris jonathannorris marked this pull request as draft June 8, 2026 14:38
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