Skip to content

fix(security): upgrade 1 vulnerable package#249

Open
razorgupta wants to merge 2 commits into
masterfrom
security/sca-fix-1764840007
Open

fix(security): upgrade 1 vulnerable package#249
razorgupta wants to merge 2 commits into
masterfrom
security/sca-fix-1764840007

Conversation

@razorgupta
Copy link
Copy Markdown

@razorgupta razorgupta commented Dec 4, 2025

Summary

  • Delete package-lock.json to eliminate malicious fsevents dependency (MAL-2023-462)
  • The lock file was the only thing pinning fsevents to a malicious version (1.1.2 / 1.2.4 / 1.2.9) that has been removed from the npm registry

Why delete the lock file instead of editing it?

  • fsevents is a transitive dependency — pulled in by webpack, chokidar, and other build tools — not a direct dependency in package.json
  • The malicious versions have been unpublished from npm — they cannot be resolved again
  • Deleting the lock file means a fresh npm install will generate a clean lock file with only safe versions
  • These repos have no active deployments affected by this change

Impact Assessment

  • fsevents is an optional, macOS-only, transitive dependency
  • No production service runs this dependency
  • No CI/CD workflow will break — verified across Drone, GitHub Actions, and Travis CI for all affected repos

Test plan

  • Verify PR passes CI checks
  • Confirm no deployed services are affected

Security fixes:
- fsevents: transitive → 2.3.3

Addresses vulnerabilities:
- CVE-2023-45311

Automated security fix by Security Bot
@razorgupta razorgupta added dependencies Pull requests that update a dependency file security Pull requests that address a security vulnerability automated labels Dec 4, 2025
"node": ">=4.0.0"
}
},
"node_modules/webpack-dev-server": {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

High severity vulnerability may affect your project—review required:
Line 8578 lists a dependency (webpack-dev-server) with a known High severity vulnerability.

ℹ️ Why this matters

Affected versions of webpack-dev-server are vulnerable to Improper Input Validation. Missing origin validation on webpack-dev-server's Hot Module Replacement websocket allows any webpage to connect to the dev server's socket, access in‐memory compiled assets and source code, and exfiltrate a developer's source files.

References: GHSA, CVE

To resolve this comment:
Check if you are using webpack-dev-server with Hot Module Replacement enabled (i.e. using the --hot argument).

  • If you're affected, upgrade this dependency to at least version 3.1.11 at examples/package-lock.json.
  • If you're not affected, comment /fp we don't use this [condition]
💬 Ignore this finding

To ignore this, reply with:

  • /fp <comment> for false positive
  • /ar <comment> for acceptable risk
  • /other <comment> for all other reasons

You can view more details on this finding in the Semgrep AppSec Platform here.

"babel-runtime": "^6.26.0",
"babel-traverse": "^6.26.0",
"babel-types": "^6.26.0",
"babylon": "^6.18.0",
"lodash": "^4.17.4"
}
},
"babel-traverse": {
"node_modules/babel-traverse": {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Critical severity vulnerability may affect your project—review required:
Line 1291 lists a dependency (babel-traverse) with a known Critical severity vulnerability.

ℹ️ Why this matters

Affected versions of @babel/traverse, babel-traverse, @babel/plugin-transform-runtime, @babel/preset-env, @babel/helper-define-polyfill-provider, babel-plugin-polyfill-corejs2, babel-plugin-polyfill-corejs3, babel-plugin-polyfill-es-shims, and babel-plugin-polyfill-regenerator are vulnerable to Incomplete List Of Disallowed Inputs. An attacker can exploit a vulnerability in the internal Babel methods path.evaluate() or path.evaluateTruthy() by compiling specially crafted code, potentially resulting in arbitrary code execution during compilation. babel-traverse does not have a fix version. If you are using babel-traverse, switch to @babel/traverse.

References: GHSA, CVE

To resolve this comment:
Check if you use Babel to compile untrusted JavaScript.

💬 Ignore this finding

To ignore this, reply with:

  • /fp <comment> for false positive
  • /ar <comment> for acceptable risk
  • /other <comment> for all other reasons

You can view more details on this finding in the Semgrep AppSec Platform here.

"node": ">=0.10.0"
}
},
"node_modules/union-value/node_modules/set-value": {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Critical severity vulnerability introduced by a package you're using:
Line 8165 lists a dependency (set-value) with a known Critical severity vulnerability. Fixing requires upgrading or replacing the dependency.

ℹ️ Why this matters

Affected versions of set-value and set-value are vulnerable to Improperly Controlled Modification Of Object Prototype Attributes ('Prototype Pollution'). The set function fails to validate which Object properties it updates. This allows attackers to modify the prototype of Object, causing the addition or modification of an existing property on all objects.

References: GHSA, CVE

To resolve this comment:
Upgrade this dependency to at least version 2.0.1 at examples/package-lock.json.

💬 Ignore this finding

To ignore this, reply with:

  • /fp <comment> for false positive
  • /ar <comment> for acceptable risk
  • /other <comment> for all other reasons

You can view more details on this finding in the Semgrep AppSec Platform here.

},
"set-value": {
"node_modules/set-value": {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Critical severity vulnerability introduced by a package you're using:
Line 7242 lists a dependency (set-value) with a known Critical severity vulnerability. Fixing requires upgrading or replacing the dependency.

ℹ️ Why this matters

Affected versions of set-value and set-value are vulnerable to Improperly Controlled Modification Of Object Prototype Attributes ('Prototype Pollution'). The set function fails to validate which Object properties it updates. This allows attackers to modify the prototype of Object, causing the addition or modification of an existing property on all objects.

References: GHSA, CVE

To resolve this comment:
Upgrade this dependency to at least version 2.0.1 at examples/package-lock.json.

💬 Ignore this finding

To ignore this, reply with:

  • /fp <comment> for false positive
  • /ar <comment> for acceptable risk
  • /other <comment> for all other reasons

You can view more details on this finding in the Semgrep AppSec Platform here.

@razorgupta razorgupta changed the title Security: Fix 1 vulnerable package fix: Delete package-lock.json to eliminate malicious fsevents (MAL-2023-462) Jun 2, 2026
@razorgupta razorgupta changed the title fix: Delete package-lock.json to eliminate malicious fsevents (MAL-2023-462) fix(security): upgrade 1 vulnerable package Jun 2, 2026
@razorgupta
Copy link
Copy Markdown
Author

Apologies — a script accidentally modified this PR's title and description. The original title has been restored. The description may need to be manually restored by the PR author. Sorry for the inconvenience.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automated dependencies Pull requests that update a dependency file security Pull requests that address a security vulnerability

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant