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

🛠 Repo: Several packages reported by npm audit #5070

Open
5 tasks done
JoshuaKGoldberg opened this issue Dec 28, 2023 · 2 comments
Open
5 tasks done

🛠 Repo: Several packages reported by npm audit #5070

JoshuaKGoldberg opened this issue Dec 28, 2023 · 2 comments
Assignees
Labels
area: security involving vulnerabilities status: accepting prs Mocha can use your help with this one!

Comments

@JoshuaKGoldberg
Copy link
Member

Tooling Suggestion Checklist

Overview

Running npm audit reports 58 vulnerabilities (34 moderate, 23 high, 1 critical):

  • @babel/traverse <7.23.2
  • axios 0.8.1 - 1.5.1
  • browserify-sign 2.6.0 - 4.2.1
  • debug <=2.6.8
  • engine.io 5.1.0 - 6.4.1
  • get-func-name <2.0.1
  • got <=11.8.3
  • http-cache-semantics <4.1.1
  • liquidjs <10.0.0
  • markdown-it <12.3.2
  • ms <2.0.0
  • nth-check <2.0.1
  • nunjucks <3.2.4
  • postcss <8.4.31
  • request *
  • semver <=5.7.1 || 6.0.0 - 6.3.0 || 7.0.0 - 7.5.1
  • semver-regex <=3.1.3
  • socket.io-parser 4.0.4 - 4.2.2
  • taffydb *
  • tough-cookie <4.1.3
  • trim-newlines <3.0.1
  • word-wrap <1.2.4

After running npm audit fix locally, npm audit reports 50 vulnerabilities (30 moderate, 20 high):

  • axios 0.8.1 - 1.5.1
  • debug <=2.6.8
  • got <=11.8.3
  • http-cache-semantics <4.1.1
  • liquidjs <10.0.0
  • markdown-it <12.3.2
  • ms <2.0.0
  • nth-check <2.0.1
  • postcss <8.4.31
  • request *
  • semver-regex <=3.1.3
  • taffydb *
  • tough-cookie <4.1.3
  • trim-newlines <3.0.1

Additional Info

It's the nature of package vulnerability alerts that most or all of these are false flags. But it's good practice to stay up-to-date just in case.

@JoshuaKGoldberg JoshuaKGoldberg added status: accepting prs Mocha can use your help with this one! area: security involving vulnerabilities labels Dec 28, 2023
@JoshuaKGoldberg JoshuaKGoldberg self-assigned this Dec 28, 2023
@JoshuaKGoldberg
Copy link
Member Author

JoshuaKGoldberg commented Dec 28, 2023

#5071 gets most of this. After it the audit report is just 5 vulnerabilities (4 moderate, 1 high):

# npm audit report

request  *
Severity: moderate
Server-Side Request Forgery in Request - https://github.com/advisories/GHSA-p8p7-x288-28g6
Depends on vulnerable versions of tough-cookie
No fix available
node_modules/request
  coveralls  *
  Depends on vulnerable versions of request
  node_modules/coveralls
  hyperlink  *
  Depends on vulnerable versions of request
  node_modules/hyperlink

taffydb  *
Severity: high
TaffyDB can allow access to any data items in the DB - https://github.com/advisories/GHSA-mxhp-79qh-mcx6
No fix available
node_modules/taffydb

tough-cookie  <4.1.3
Severity: moderate
tough-cookie Prototype Pollution vulnerability - https://github.com/advisories/GHSA-72xf-g2v4-qvf3
No fix available
node_modules/request/node_modules/tough-cookie

5 vulnerabilities (4 moderate, 1 high)

Some issues need review, and may require choosing
a different dependency.

Looking at the three relevant packages:

@voxpelli
Copy link
Member

Related:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: security involving vulnerabilities status: accepting prs Mocha can use your help with this one!
Projects
None yet
Development

No branches or pull requests

2 participants