Skip to content

Fix dependency security vulnerabilities#2306

Merged
MarceloRGonc merged 7 commits into
mainfrom
dependabot/update-dependencies
May 26, 2026
Merged

Fix dependency security vulnerabilities#2306
MarceloRGonc merged 7 commits into
mainfrom
dependabot/update-dependencies

Conversation

@MarceloRGonc
Copy link
Copy Markdown
Contributor

@MarceloRGonc MarceloRGonc commented May 25, 2026

Fixes OPS-4380

Direct dependency updates:

  • uuid: 10.0.0 → 11.1.1 (fixes medium-severity buffer bounds check issue)
  • @playwright/test: 1.52.0 → 1.60.0 (fixes high-severity SSL cert verification bypass)
  • postcss-url: 8.0.0 → 10.1.4 (fixes 2 moderate postcss vulnerabilities)

Removed unnecessary dependency:

  • npm: removed from package.json — the Dockerfile and CI use the system npm bundled with Node.js. This eliminates 3 vulnerabilities (brace-expansion, picomatch, ip-address) that were bundled inside it.

Added npm overrides for transitive vulnerabilities:

  • js-cookie: forced to 3.0.7 (high-severity prototype hijack, transitive from @segment/analytics-next and react-use)
  • ws: forced to 8.20.1 (medium-severity memory disclosure, transitive from socket.io/jsdom)
  • axios: forced to root version via $axios (medium-severity CRLF injection, transitive from nx)
  • uuid: forced to root version via $uuid (ensures all transitive copies use patched 11.1.1)

@linear
Copy link
Copy Markdown

linear Bot commented May 25, 2026

OPS-4380

@MarceloRGonc MarceloRGonc changed the title WIP Fix dependency security vulnerabilities May 25, 2026
@MarceloRGonc MarceloRGonc marked this pull request as ready for review May 25, 2026 16:53
Copilot AI review requested due to automatic review settings May 25, 2026 16:53
@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates JavaScript dependencies to address reported security vulnerabilities (OPS-4380) by bumping several direct deps, removing the npm package dependency, and enforcing patched versions of select transitive packages via npm overrides.

Changes:

  • Bumped direct dependencies: uuid, @playwright/test (and playwright), and postcss-url.
  • Removed the npm package dependency from package.json / package-lock.json.
  • Added npm overrides to force patched transitive versions (js-cookie, ws) and align transitive axios/uuid with root versions.

Reviewed changes

Copilot reviewed 1 out of 3 changed files in this pull request and generated 2 comments.

File Description
package.json Removes npm, bumps direct deps, and adds overrides for vulnerable transitive packages.
package-lock.json Regenerates lockfile to reflect removed npm, upgraded direct deps, and applied override resolutions.

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

Comment thread package.json
"rollup": "npm:@rollup/wasm-node"
}
},
"js-cookie": "3.0.7",
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Our codebase never uses useCookie from react-use. It only imports:

  • useEffectOnce
  • useDeepCompareEffect
  • usePrevious
  • useMeasure
  • useUpdateEffect

None of these touch js-cookie. The js-cookie dependency in react-use is only used by its useCookie hook, which we don't import.

Comment thread package.json
Comment on lines 243 to 246
"node-cron": "3.0.3",
"node-emoji": "2.2.0",
"nodemailer": "8.0.5",
"npm": "10.9.8",
"nx-cloud": "19.1.3",
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I will handle it in another PR

@MarceloRGonc MarceloRGonc merged commit 5584520 into main May 26, 2026
21 checks passed
@MarceloRGonc MarceloRGonc deleted the dependabot/update-dependencies branch May 26, 2026 13:57
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