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

Update dependencies version to fix security issues #37

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pmprete
Copy link

@pmprete pmprete commented Apr 30, 2020

Bump the version of mocha, yargs and mkdirp to solve the issues shown by npm audit:

npm audit security report

Run npm install [email protected] to resolve 1 vulnerability

SEMVER WARNING: Recommended action is a potentially breaking change
│ Low │ Denial of Service
│ Package │ mem
│ Dependency of │ yargs
│ More info │ https://npmjs.com/advisories/1084

Run npm install [email protected] to resolve 1 vulnerability

SEMVER WARNING: Recommended action is a potentially breaking change
│ Low │ Prototype Pollution
│ Package │ minimist
│ Dependency of │ mocha
│ Path │ mocha > mkdirp > minimist
│ More info │ https://npmjs.com/advisories/1179

Run npm install [email protected] to resolve 1 vulnerability

SEMVER WARNING: Recommended action is a potentially breaking change
│ Low │ Prototype Pollution
│ Package │ minimist
│ Dependency of │ mkdirp
│ Path │ mkdirp > minimist
│ More info │ https://npmjs.com/advisories/1179

found 3 low severity vulnerabilities in 88 scanned packages
3 vulnerabilities require semver-major dependency updates.

Bump the version of mocha, yargs and mkdirp
@duaraghav8
Copy link
Owner

duaraghav8 commented May 4, 2020

@pmprete thanks! I'll test it out locally and then merge

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

Successfully merging this pull request may close these issues.

2 participants