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

Security vulnerabilities in generator-loopback #352

Closed
MelleB opened this issue Jun 21, 2018 · 3 comments
Closed

Security vulnerabilities in generator-loopback #352

MelleB opened this issue Jun 21, 2018 · 3 comments
Assignees

Comments

@MelleB
Copy link

MelleB commented Jun 21, 2018

nsp check fails on the [email protected] dependency.
Repository of the open package suggests to migrate to opn

Description/Steps to reproduce

$ yarn run nsp check
┌───────────────┬─────────────────────────────────────────────────────────────────────────────────┐
│               │ Command Injection                                                               │
├───────────────┼─────────────────────────────────────────────────────────────────────────────────┤
│ Name          │ open                                                                            │
├───────────────┼─────────────────────────────────────────────────────────────────────────────────┤
│ CVSS          │ 10 (Critical)                                                                   │
├───────────────┼─────────────────────────────────────────────────────────────────────────────────┤
│ Installed     │ 0.0.5                                                                           │
├───────────────┼─────────────────────────────────────────────────────────────────────────────────┤
│ Vulnerable    │ All                                                                             │
├───────────────┼─────────────────────────────────────────────────────────────────────────────────┤
│ Patched       │ None                                                                            │
├───────────────┼─────────────────────────────────────────────────────────────────────────────────┤
│ Path          │ [email protected] > [email protected] > [email protected] > [email protected]     │
├───────────────┼─────────────────────────────────────────────────────────────────────────────────┤
│ More Info     │ https://nodesecurity.io/advisories/663                                          │
└───────────────┴─────────────────────────────────────────────────────────────────────────────────┘
┌───────────────┬─────────────────────────────────────────────────────────────────────────────────┐
│               │ Prototype pollution attack                                                      │
├───────────────┼─────────────────────────────────────────────────────────────────────────────────┤
│ Name          │ hoek                                                                            │
├───────────────┼─────────────────────────────────────────────────────────────────────────────────┤
│ CVSS          │ 4 (Medium)                                                                      │
├───────────────┼─────────────────────────────────────────────────────────────────────────────────┤
│ Installed     │ 2.16.3                                                                          │
├───────────────┼─────────────────────────────────────────────────────────────────────────────────┤
│ Vulnerable    │ <= 4.2.0 || >= 5.0.0 < 5.0.3                                                    │
├───────────────┼─────────────────────────────────────────────────────────────────────────────────┤
│ Patched       │ > 4.2.0 < 5.0.0 || >= 5.0.3                                                     │
├───────────────┼─────────────────────────────────────────────────────────────────────────────────┤
│ Path          │ [email protected] > [email protected] > [email protected] > zosconnect-no… │
├───────────────┼─────────────────────────────────────────────────────────────────────────────────┤
│ More Info     │ https://nodesecurity.io/advisories/566                                          │
└───────────────┴─────────────────────────────────────────────────────────────────────────────────┘
┌───────────────┬─────────────────────────────────────────────────────────────────────────────────┐
│               │ Prototype Pollution                                                             │
├───────────────┼─────────────────────────────────────────────────────────────────────────────────┤
│ Name          │ lodash                                                                          │
├───────────────┼─────────────────────────────────────────────────────────────────────────────────┤
│ CVSS          │ 2 (Low)                                                                         │
├───────────────┼─────────────────────────────────────────────────────────────────────────────────┤
│ Installed     │ 3.10.1                                                                          │
├───────────────┼─────────────────────────────────────────────────────────────────────────────────┤
│ Vulnerable    │ <4.17.5                                                                         │
├───────────────┼─────────────────────────────────────────────────────────────────────────────────┤
│ Patched       │ >=4.17.5                                                                        │
├───────────────┼─────────────────────────────────────────────────────────────────────────────────┤
│ Path          │ [email protected] > [email protected] > [email protected] > yeoman-genera… │
├───────────────┼─────────────────────────────────────────────────────────────────────────────────┤
│ More Info     │ https://nodesecurity.io/advisories/577                                          │
└───────────────┴─────────────────────────────────────────────────────────────────────────────────┘
┌───────────────┬─────────────────────────────────────────────────────────────────────────────────┐
│               │ Prototype Pollution                                                             │
├───────────────┼─────────────────────────────────────────────────────────────────────────────────┤
│ Name          │ deep-extend                                                                     │
├───────────────┼─────────────────────────────────────────────────────────────────────────────────┤
│ CVSS          │ 2 (Low)                                                                         │
├───────────────┼─────────────────────────────────────────────────────────────────────────────────┤
│ Installed     │ 0.4.2                                                                           │
├───────────────┼─────────────────────────────────────────────────────────────────────────────────┤
│ Vulnerable    │ <=0.5.0                                                                         │
├───────────────┼─────────────────────────────────────────────────────────────────────────────────┤
│ Patched       │ >=0.5.1                                                                         │
├───────────────┼─────────────────────────────────────────────────────────────────────────────────┤
│ Path          │ [email protected] > [email protected] > [email protected] > yeoman-genera… │
├───────────────┼─────────────────────────────────────────────────────────────────────────────────┤
│ More Info     │ https://nodesecurity.io/advisories/612                                          │
└───────────────┴─────────────────────────────────────────────────────────────────────────────────┘
@dhmlau
Copy link
Member

dhmlau commented Jun 21, 2018

@virkt25 @bajtos , could you PTAL?

@bajtos
Copy link
Member

bajtos commented Jun 22, 2018

Thank you @MelleB for reporting this problem and offering a better alternative to open.

I opened a pull request to upgrade from open to opn, see #353

The prototype-pollution issues in lodash coming through yeoman-generator will be difficult to address. We use yeoman-generator in a way that's not very well supported these days, upgrades are cumbersome. Considering Low severity of this vulnerability, I am not sure if we will find time to upgrade yeoman-generator anytime soon.

The last remaining security vulnerability is coming through zosconnect-node, which is maintained by a different IBM team. I'll work with them to get this addressed, see zosconnect/zosconnect-node#27

@bajtos
Copy link
Member

bajtos commented Jun 22, 2018

The prototype-pollution issues in lodash coming through yeoman-generator will be difficult to address. We use yeoman-generator in a way that's not very well supported these days, upgrades are cumbersome. Considering Low severity of this vulnerability, I am not sure if we will find time to upgrade yeoman-generator anytime soon.

I opened a new issue to keep track of this work, see #355

open and hoek vulnerabilities have been fixed by dependency updates

$ npm i generator-loopback
(...)
+ [email protected]
added 783 packages from 894 contributors and audited 5482 packages in 39.897s
found 2 low severity vulnerabilities
  run `npm audit fix` to fix them, or `npm audit` for details

@bajtos bajtos closed this as completed Jun 22, 2018
@bajtos bajtos changed the title nsp security check fails (1 critical, 1 medium, 2 low) Fix high-severity security vulnerabilities in generator-loopback Jun 22, 2018
@bajtos bajtos changed the title Fix high-severity security vulnerabilities in generator-loopback Security vulnerabilities in generator-loopback Jun 22, 2018
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