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.5 #21

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

Conversation

Maelstrom96
Copy link
Member

This PR was automatically created by Snyk using the credentials of a real user.


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 Priority Score (*) Issue Breaking Change Exploit Maturity
high severity 696/1000
Why? Proof of Concept exploit, Has a fix available, CVSS 7.5
Regular Expression Denial of Service (ReDoS)
SNYK-JS-ANSIREGEX-1583908
No Proof of Concept

(*) Note that the real score may have changed since the PR was raised.

Commit messages
Package name: sails The new version differs by 162 commits.
  • 617674f 1.2.5
  • 2ca4923 Update sails-generate dependency
  • bab841e [fixes #6965] Display underlying `raw` error on fail to lift app (#6966)
  • 9af6826 add React Native & phonegap, remove some old things
  • 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

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


Learn how to fix vulnerabilities with free interactive lessons:

🦉 Regular Expression Denial of Service (ReDoS)

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