Npm-check-updates: 2 Vulnerabilities Found: Prototype pollution attack

Created on 7 Apr 2018  Â·  8Comments  Â·  Source: raineorshine/npm-check-updates

  • [✓] I have read the list of known issues before filing this issue.
  • [✓] I have searched for similiar issues before filing this issue.

  • node version: v8.11.1

  • npm version: v5.8.0
  • npm-check-updates version: v2.14.1

  • Operating system/terminal environment: OSX 10.13.3 / Terminal & iTerm
  • Command you ran:

    • Install global nsp: npm i -g nsp

    • cd into the npm-check-updates module folder: cd [...]/.nvm/versions/node/v8.11.1/lib/node_modules/npm-check-updates

    • Run security check: nsp check

Expected behaviour

(+) No known vulnerabilities found

Actual behaviour

(+) 2 vulnerabilities found

screenshot

Prototype pollution attack
More Info │ https://nodesecurity.io/advisories/566

Steps to reproduce

  • Install global nsp: npm i -g nsp
  • cd into the npm-check-updates module folder: cd [...]/.nvm/versions/node/v8.11.1/lib/node_modules/npm-check-updates
  • Run security check: nsp check
bug

Most helpful comment

@raineorshine I'm currently working on a fork and refactoring.

All 8 comments

npm audit reported 80 vulnerabilities with npm-check-updates v2.14.2 for me.

This is the output: audit.txt

I ran ncu -a on this repo and installed modules. Only three vulnerabilities came back and they were low priority. Why are the packages this repo uses so far out of date?

@NetOperatorWibby I have had to focus on other projects recently. Unfortunately nobody else has contributed in a long time. ncu has a lot of users, so hopefully someone will step up. It would help a lot of people.

@raineorshine I'm currently working on a fork and refactoring.

@NetOperatorWibby Wonderful. It would be so great to incorporate your changes back into the source.

@raineorshine Seems like people have been trying to help via PRs but nothing's merged.

@NetOperatorWibby A few PR's over the last 3 years. The unmerged PR's are either waiting for the v3 milestone or needed additional work.

I've abandoned my fork and started using https://www.npmjs.com/package/updates. It has less dependencies and similar usage. Still, npm-check-updates has served me well in the past, thanks for working on it.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

benface picture benface  Â·  7Comments

ibc picture ibc  Â·  7Comments

Simek picture Simek  Â·  5Comments

vayurobins picture vayurobins  Â·  5Comments

evgenyrodionov picture evgenyrodionov  Â·  4Comments