Skip to content
This repository has been archived by the owner on Jan 24, 2022. It is now read-only.

@openzeppelin/upgrades package dependency vulnerabilities #1514

Open
ben-kaufman opened this issue Mar 23, 2020 · 2 comments
Open

@openzeppelin/upgrades package dependency vulnerabilities #1514

ben-kaufman opened this issue Mar 23, 2020 · 2 comments
Labels
kind:bug topic:upgrades.js Related to OpenZeppelin Upgrades

Comments

@ben-kaufman
Copy link

Hi, when adding the @openzeppelin/upgrades package to my project I receive warnings on package vulnerabilities (running npm audit provides detail). They all seem to arise from the web3 version you use here. Updating the web3 package used here should solve the issue.

@abcoathup
Copy link
Contributor

Hi @ben-kaufman!

We have been able to reproduce this issue by following these steps:

$ mkdir issue1514 && cd issue1514
$ npm init -y
$ npm i @openzeppelin/upgrades@rc
$ npm audit

                       === npm audit security report ===

┌──────────────────────────────────────────────────────────────────────────────┐
│                                Manual Review                                 │
│            Some vulnerabilities require your attention to resolve            │
│                                                                              │
│         Visit https://go.npm.me/audit-guide for additional guidance          │
└──────────────────────────────────────────────────────────────────────────────┘
┌───────────────┬──────────────────────────────────────────────────────────────┐
│ Low           │ Insecure Credential Storage                                  │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Package       │ web3                                                         │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Patched in    │ No patch available                                           │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Dependency of │ @openzeppelin/upgrades                                       │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Path          │ @openzeppelin/upgrades > web3                                │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ More info     │ https://npmjs.com/advisories/877                             │
└───────────────┴──────────────────────────────────────────────────────────────┘
┌───────────────┬──────────────────────────────────────────────────────────────┐
│ High          │ Arbitrary File Write                                         │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Package       │ decompress                                                   │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Patched in    │ No patch available                                           │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Dependency of │ @openzeppelin/upgrades                                       │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Path          │ @openzeppelin/upgrades > web3 > web3-bzz > swarm-js >        │
│               │ decompress                                                   │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ More info     │ https://npmjs.com/advisories/1217                            │
└───────────────┴──────────────────────────────────────────────────────────────┘
found 2 vulnerabilities (1 low, 1 high) in 36898 scanned packages
  2 vulnerabilities require manual review. See the full report for details.

Thanks so much for reporting it! The project owner will review and triage this issue during the next week.

@abcoathup
Copy link
Contributor

The vulnerabilities are in web3 or it's dependencies and are not fixed in the latest version of web3 (You can check by installing web3@1.2.6).
#700 (comment)

The Arbitrary File Write vulnerability should be fixed in a future version of web3
web3/web3.js#3399

@frangio frangio added the topic:upgrades.js Related to OpenZeppelin Upgrades label May 26, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
kind:bug topic:upgrades.js Related to OpenZeppelin Upgrades
Projects
None yet
Development

No branches or pull requests

3 participants