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

Fix some dependency vulnerabilities #2

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

wdoug
Copy link

@wdoug wdoug commented Nov 22, 2019

After running npm install with node 10, npm reported a bunch of security vulnerabilities in the installed dependencies:
Screen Shot 2019-11-22 at 2 07 19 PM
Running npm audit fix fixed most of the issues and things still seem to work fine. The remaining 60 vulnerabilities require upgrading dependencies with breaking changes:

found 75 vulnerabilities (65 low, 2 moderate, 8 high) in 36983 scanned packages
  73 vulnerabilities require semver-major dependency updates.
  2 vulnerabilities require manual review. See the full report for details.

When I tried upgrading those dependencies things broke so they will take a little bit more work to fix. Here is a pull request with the first autofixed vulnerabilities to start.

dcolthorp pushed a commit that referenced this pull request Apr 1, 2020
Check in auto-updated package-lock.json
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.

3 participants