Skip to content
This repository has been archived by the owner on Aug 3, 2023. It is now read-only.

npm install: 3 high severity vulnerabilities #1758

Closed
pyk opened this issue Feb 6, 2021 · 1 comment · Fixed by #1780
Closed

npm install: 3 high severity vulnerabilities #1758

pyk opened this issue Feb 6, 2021 · 1 comment · Fixed by #1780
Labels
onboarding Issues with installation or login

Comments

@pyk
Copy link

pyk commented Feb 6, 2021

Vulnerabilites report from npm install:

$ npm i @cloudflare/wrangler -g

added 34 packages, and audited 35 packages in 14s

3 high severity vulnerabilities

To address all issues, run:
  npm audit fix

Run `npm audit` for details.

node & npm version

$ node --version
v15.5.0
$ npm --version
7.3.0
@ags799
Copy link
Contributor

ags799 commented Feb 10, 2021

You can see the details if you run npm audit from within the npm directory of the project:

# npm audit report

axios  <0.21.1
Severity: high
Server-Side Request Forgery - https://npmjs.com/advisories/1594
fix available via `npm audit fix --force`
Will install [email protected], which is a breaking change
node_modules/axios
  binary-install  <=0.1.0
  Depends on vulnerable versions of axios
  node_modules/binary-install

npm  <=6.14.5
Severity: high
Symlink reference outside of node_modules - https://npmjs.com/advisories/1436
Global node_modules Binary Overwrite - https://npmjs.com/advisories/1437
Sensitive Data Exposure - https://npmjs.com/advisories/1543
npm Token Leak - https://npmjs.com/advisories/98
fix available via `npm audit fix`


3 high severity vulnerabilities

To address issues that do not require attention, run:
  npm audit fix

To address all issues (including breaking changes), run:
  npm audit fix --force

I don't think we need to worry about the SSRF in axios. Not sure about the npm vulnerabilities either.

Note that I ran this with my own npm at 7.5.1.

@Electroid Electroid added the onboarding Issues with installation or login label Feb 22, 2021
@xortive xortive mentioned this issue Feb 25, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
onboarding Issues with installation or login
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants