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] Security upgrade sails from 1.0.2 to 1.2.4 #6

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.

As this is a private repository, Snyk-bot does not have access. Therefore, this PR has been created automatically, but appears to have been created by a real user.

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
medium severity Prototype Pollution
SNYK-JS-MINIMIST-559764
No Proof of Concept
Commit messages
Package name: sails The new version differs by 158 commits.
  • db197e3 1.2.4
  • eac1605 Merge pull request #6963 from ebaynaud/MinimistUpgrade
  • 0d3c5f8 Upgrade minimist
  • 486df99 Fix table formatting in README
  • 684329a Merge pull request #6947 from Yohanna/chore/update-readme
  • d367019 Remove TingoDB adapter from README
  • ca8a2f6 Fix typo in error message
  • c7d62fd Merge pull request #6928 from balderdashy/readme-update
  • a5e160e Name update
  • ef61528 team information update as requested
  • 64a77e6 Fix typo in log
  • 01100a1 minor comment fix
  • 40802be Correct comments in `register-action.js` to account for the fact that we're now exposing the `force` argument
  • 9169cbf minor text fixes
  • d0e596e Make sails.getRouteFor() usage error message a bit clearer
  • 51599b4 Update error message in `sails.getRouteFor()` to keep the usage examples consistent, and other minor text fixes
  • 8479036 Tidy up comment for `loadActionModules()`
  • 53d0473 Merge pull request #6543 from travispwingo/master
  • d2bd6db Tidy up some comments
  • b47d465 Merge pull request #6855 from balderdashy/csrf-disabling-for-regex-route
  • f7a0f04 IN CSRF hook, only lowercase the route path if the route isn't a regex
  • 8242374 Merge pull request #6847 from balderdashy/destroy-blueprint-action
  • ba450bf Update destroy blueprint action so that `afterDestroy` lifecycle callbacks will still run by default
  • eea8df4 Merge pull request #6825 from Tarrask/patch-1

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