Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

npm-run-all2 is reported as having a moderate severity vulnerabilty #113

Closed
langthiennhai opened this issue Jul 3, 2023 · 2 comments · Fixed by #114
Closed

npm-run-all2 is reported as having a moderate severity vulnerabilty #113

langthiennhai opened this issue Jul 3, 2023 · 2 comments · Fixed by #114

Comments

@langthiennhai
Copy link

As of today (03 July 2023), running npm audit on a project that uses npm-run-all2 results in the following audit report:

npm audit report

semver <7.5.2
Severity: moderate
semver vulnerable to Regular Expression Denial of Service - GHSA-c2qf-rxjj-qqgw
No fix available
node_modules/eslint-plugin-import/node_modules/semver
node_modules/semver
eslint-plugin-import >=2.27.4
Depends on vulnerable versions of semver
node_modules/eslint-plugin-import
normalize-package-data <=2.5.0
Depends on vulnerable versions of semver
node_modules/normalize-package-data
read-pkg <=5.2.0
Depends on vulnerable versions of normalize-package-data
node_modules/read-pkg
npm-run-all2 *
Depends on vulnerable versions of read-pkg
node_modules/npm-run-all2

Trying npm audit fix --force does not work, at least not for me.

A fix for semver is available: https://github.com/npm/node-semver/releases/tag/v7.5.3

Please update npm-run-all's dependency tree to address this vulnerability.

@bcomnes
Copy link
Owner

bcomnes commented Jul 3, 2023

Need to go through and do some updates to some packages that went esm only. Unfortunately this is non-trivial, so I haven't had time. A PR would be appreciated here if you need this asap. The vulns are not an issue for these us cases however.

@bcomnes
Copy link
Owner

bcomnes commented Jul 4, 2023

Ok, I updated the esm only deps in #114 Will be out in the next release.

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 a pull request may close this issue.

2 participants