Skip to content

NPM audit reveals 9 moderate vulnerabilities in dependencies #1499

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

Closed
juliedavila opened this issue May 17, 2018 · 2 comments
Closed

NPM audit reveals 9 moderate vulnerabilities in dependencies #1499

juliedavila opened this issue May 17, 2018 · 2 comments

Comments

@juliedavila
Copy link

There are 9 but 8 of them are from the deps of node-pre-gyp that depend on hoek annpm audit nodegit reveals:

                                                                                
                       === npm audit security report ===                        
                                                                                



┌──────────────────────────────────────────────────────────────────────────────┐
│                                Manual Review                                 │
│            Some vulnerabilities require your attention to resolve            │
│                                                                              │
│         Visit https://go.npm.me/audit-guide for additional guidance          │
└──────────────────────────────────────────────────────────────────────────────┘
┌───────────────┬──────────────────────────────────────────────────────────────┐
│ moderate      │ Prototype pollution                                          │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Package       │ hoek                                                         │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Dependency of │ nodegit                                                      │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Path          │ nodegit > node-pre-gyp > hawk > boom > hoek                  │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ More info     │ https://nodesecurity.io/advisories/566                       │
└───────────────┴──────────────────────────────────────────────────────────────┘
┌───────────────┬──────────────────────────────────────────────────────────────┐
│ moderate      │ Prototype pollution                                          │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Package       │ hoek                                                         │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Dependency of │ nodegit                                                      │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Path          │ nodegit > node-pre-gyp > hawk > cryptiles > boom > hoek      │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ More info     │ https://nodesecurity.io/advisories/566                       │
└───────────────┴──────────────────────────────────────────────────────────────┘
┌───────────────┬──────────────────────────────────────────────────────────────┐
│ moderate      │ Prototype pollution                                          │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Package       │ hoek                                                         │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Dependency of │ nodegit                                                      │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Path          │ nodegit > node-pre-gyp > hawk > hoek                         │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ More info     │ https://nodesecurity.io/advisories/566                       │
└───────────────┴──────────────────────────────────────────────────────────────┘
┌───────────────┬──────────────────────────────────────────────────────────────┐
│ moderate      │ Prototype pollution                                          │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Package       │ hoek                                                         │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Dependency of │ nodegit                                                      │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Path          │ nodegit > node-pre-gyp > hawk > sntp > hoek                  │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ More info     │ https://nodesecurity.io/advisories/566                       │
└───────────────┴──────────────────────────────────────────────────────────────┘
┌───────────────┬──────────────────────────────────────────────────────────────┐
│ moderate      │ Prototype pollution                                          │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Package       │ hoek                                                         │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Dependency of │ nodegit                                                      │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Path          │ nodegit > node-pre-gyp > request > hawk > boom > hoek        │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ More info     │ https://nodesecurity.io/advisories/566                       │
└───────────────┴──────────────────────────────────────────────────────────────┘
┌───────────────┬──────────────────────────────────────────────────────────────┐
│ moderate      │ Prototype pollution                                          │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Package       │ hoek                                                         │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Dependency of │ nodegit                                                      │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Path          │ nodegit > node-pre-gyp > request > hawk > cryptiles > boom > │
│               │ hoek                                                         │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ More info     │ https://nodesecurity.io/advisories/566                       │
└───────────────┴──────────────────────────────────────────────────────────────┘
┌───────────────┬──────────────────────────────────────────────────────────────┐
│ moderate      │ Prototype pollution                                          │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Package       │ hoek                                                         │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Dependency of │ nodegit                                                      │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Path          │ nodegit > node-pre-gyp > request > hawk > hoek               │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ More info     │ https://nodesecurity.io/advisories/566                       │
└───────────────┴──────────────────────────────────────────────────────────────┘
┌───────────────┬──────────────────────────────────────────────────────────────┐
│ moderate      │ Prototype pollution                                          │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Package       │ hoek                                                         │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Dependency of │ nodegit                                                      │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Path          │ nodegit > node-pre-gyp > request > hawk > sntp > hoek        │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ More info     │ https://nodesecurity.io/advisories/566                       │
└───────────────┴──────────────────────────────────────────────────────────────┘
┌───────────────┬──────────────────────────────────────────────────────────────┐
│ moderate      │ Out-of-bounds Read                                           │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Package       │ stringstream                                                 │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Dependency of │ nodegit                                                      │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Path          │ nodegit > node-pre-gyp > request > stringstream              │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ More info     │ https://nodesecurity.io/advisories/664                       │
└───────────────┴──────────────────────────────────────────────────────────────┘

[!] 9 vulnerabilities found - Packages audited: 532 (0 dev, 12 optional)
    Severity: 9 moderate

I think this could be fixable by upgrading the version of node-pre-gyp in package.json as seen here mapbox/node-pre-gyp#346

@padraic-edwards
Copy link

Hi - is there any ETA for when the issues above will be addressed? Thanks.

@Croydon
Copy link
Contributor

Croydon commented Jul 27, 2018

I have updated all direct dependencies in #1519

I would rather prefer not to mess with transistive dependencies for now, but rather wait till those dependencies have updates themselves.

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

No branches or pull requests

3 participants