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

[Request] Upgrade tar to v6.2.1 #3013

Closed
anhcao142 opened this issue Apr 10, 2024 · 6 comments
Closed

[Request] Upgrade tar to v6.2.1 #3013

anhcao142 opened this issue Apr 10, 2024 · 6 comments
Labels
node-pre-gyp is unmaintained mapbox/node-pre-gyp#657

Comments

@anhcao142
Copy link

Hi currently, our yarn npm audit alerts that tar versions < 6.2.1 are vulnerable. This is our log

└─ tar
   ├─ ID: 1096915
   ├─ Issue: Denial of service while parsing a tar file due to lack of folders count validation
   ├─ URL: https://github.com/advisories/GHSA-f5x3-32g6-xq36
   ├─ Severity: moderate
   ├─ Vulnerable Versions: <6.2.1
   │
   ├─ Tree Versions
   │  └─ 6.2.0
   │
   └─ Dependents
      └─ @mapbox/node-pre-gyp@npm:1.0.11
@cclauss
Copy link
Contributor

cclauss commented Apr 10, 2024

https://www.npmjs.com/package/tar

"tar": "^6.1.2",

@anhcao142 Are you able to make a pull request to update this dependency?

@lukekarrys
Copy link
Member

@anhcao142 The log you posted indicates that @mapbox/node-pre-gyp is the dependency responsible for the vulnerable version of tar.

@anhcao142
Copy link
Author

Yeah I just noticed that, will close this now

@anhcao142 anhcao142 closed this as not planned Won't fix, can't repro, duplicate, stale Apr 11, 2024
@cclauss cclauss added the node-pre-gyp is unmaintained mapbox/node-pre-gyp#657 label Apr 11, 2024
@mvanzoest
Copy link
Contributor

Hi, just as a heads up, it looks like tar is a direct dependency of node-gyp:

image

@cclauss
Copy link
Contributor

cclauss commented Apr 15, 2024

Yes. As stated above at #3013 (comment)

@anhcao142 Are you able to make a pull request to update this dependency?

@mvanzoest
Copy link
Contributor

Yes. As stated above at #3013 (comment)

@anhcao142 Are you able to make a pull request to update this dependency?

#3021

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
node-pre-gyp is unmaintained mapbox/node-pre-gyp#657
Projects
None yet
Development

No branches or pull requests

4 participants