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

Dependency "tar" - high severity vulnerability #730

Closed
NickThompson1993 opened this issue Apr 11, 2019 · 5 comments
Closed

Dependency "tar" - high severity vulnerability #730

NickThompson1993 opened this issue Apr 11, 2019 · 5 comments

Comments

@NickThompson1993
Copy link

NickThompson1993 commented Apr 11, 2019

npm is reporting that the "tar" (gulp-sass > node-sass > node-gyp > tar) dependency version is vulnerable to Arbitrary File Overwrite. https://nodesecurity.io/advisories/803

This issue only affects tar <4.4.2, would it be possible to update Gulp-Sass's dependency to a more recent version to prevent this?

Many thanks

@stof
Copy link
Contributor

stof commented Apr 12, 2019

this is currently being worked on by the node-gyp team: nodejs/node-gyp#1713

gulp-sass does not directly uses tar, so cannot fix it itself.

@RobertAKARobin
Copy link

@stof Looks like node-gyp merged the fix.

@stof
Copy link
Contributor

stof commented Apr 12, 2019

@RobertAKARobin they currently merged it only in the master branch, which is the dev version of the upcoming 4.0.
There is no release containing the fix yet. The work on backporting it to their 3.8 branch to create a patch 3.8.1 release is in progress in nodejs/node-gyp#1718

@xzyfer
Copy link
Collaborator

xzyfer commented Apr 23, 2019

Tracking in sass/node-sass#2625.

Locking this issue in the mean time.

Repository owner locked and limited conversation to collaborators Apr 23, 2019
@xzyfer
Copy link
Collaborator

xzyfer commented May 18, 2019

node-tar released a fix. Run npm update node-tar to resolve the npm audit warnings.

@xzyfer xzyfer closed this as completed May 18, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants