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

[Snyk] Fix for 3 vulnerabilities #52

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

Conversation

snyk-bot
Copy link

Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.

Changes included in this PR

  • Changes to the following files to upgrade the vulnerable dependencies to a fixed version:
    • package.json
    • package-lock.json

Vulnerabilities that will be fixed

With an upgrade:
Severity Issue Breaking Change Exploit Maturity
high severity Arbitrary File Write via Archive Extraction (Zip Slip)
npm:adm-zip:20180415
No Mature
high severity Content & Code Injection (XSS)
npm:marked:20150520
No No Known Exploit
high severity Prototype Override Protection Bypass
npm:qs:20170213
No No Known Exploit
Commit messages
Package name: adm-zip The new version differs by 50 commits.

See the full diff

Package name: body-parser The new version differs by 221 commits.

See the full diff

Package name: express The new version differs by 231 commits.
  • d43b074 4.15.2
  • 05fd1e4 deps: update example dependencies
  • 85c96fd deps: qs@6.4.0
  • d32ed68 4.15.1
  • 57d3dfd examples: merge the jade example into ejs
  • eece385 tests: use path.join instead of concatenation
  • 8eb95ae examples: use path.join instead of concatenation
  • 67168fe deps: serve-static@1.12.1
  • c0089d9 deps: send@0.15.1
  • dc8acc8 tests: use supertest expect for simple assertions
  • 7027b37 lint: remove unused err argument
  • b4550fb Use ejs instead of jade within engine jsdoc
  • 4012846 examples: use static assets in search example
  • 6d9b127 build: Node.js@7.6
  • 504a51c 4.15.0
  • 7f96896 deps: update example dependencies
  • f59de6a build: Node.js@7.7
  • 7247554 build: Node.js@6.10
  • 146a13e build: Node.js@4.8
  • 9722202 Add next("router") to exit from router
  • 51f5290 Fix case where router.use skipped requests routes did not
  • 8b6dc6c Use "%o" in path debug to tell types apart
  • 081b811 perf: add fast match path for "*" route
  • 1f71fae tests: add lone "*" route tests

See the full diff

Package name: marked The new version differs by 5 commits.

See the full diff

Check the changes in this PR to ensure they won't cause issues with your project.


Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.

For more information:

🧐 View latest project report

🛠 Adjust project settings

📚 Read more about Snyk's upgrade and patch logic

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.

1 participant