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 4 vulnerabilities #60

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

Conversation

simha95
Copy link
Owner

@simha95 simha95 commented Aug 23, 2022

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.

Changes included in this PR

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

Vulnerabilities that will be fixed

With an upgrade:
Severity Priority Score (*) Issue Breaking Change Exploit Maturity
medium severity 586/1000
Why? Proof of Concept exploit, Has a fix available, CVSS 5.3
Regular Expression Denial of Service (ReDoS)
SNYK-JS-VALIDATOR-1090599
Yes Proof of Concept
medium severity 586/1000
Why? Proof of Concept exploit, Has a fix available, CVSS 5.3
Regular Expression Denial of Service (ReDoS)
SNYK-JS-VALIDATOR-1090600
Yes Proof of Concept
medium severity 586/1000
Why? Proof of Concept exploit, Has a fix available, CVSS 5.3
Regular Expression Denial of Service (ReDoS)
SNYK-JS-VALIDATOR-1090601
Yes Proof of Concept
medium severity 586/1000
Why? Proof of Concept exploit, Has a fix available, CVSS 5.3
Regular Expression Denial of Service (ReDoS)
SNYK-JS-VALIDATOR-1090602
Yes Proof of Concept

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

Commit messages
Package name: anchor The new version differs by 84 commits.
  • f24a4fe 1.4.1
  • 609cef2 Update isURL validation and add localhost test (req and res available in views balderdashy/sails#113)
  • 2bb793a Update validator@13.7.0 (switching to production balderdashy/sails#112)
  • f871463 1.4.0
  • 18e0e7f Add original invalid data to the error message for selected rules, in order to reduce the time it takes to debug mistakes
  • bfcd549 Minor improvement to test output.
  • e42acb7 1.3.0
  • 4e519da Nabbed one other one I missed
  • 7d77b02 Finish the rest of the TODOs began in d6f82e75d3827ceee6e0000b9e98cc752f461f51
  • f05f536 Take care of some of the TODOs from previous commit. Also add an isError check when handling errors thrown from validation rules to prevent potential issues in extreme edge cases (where something weird gets thrown)
  • d6f82e7 Add validator upgrade TODOs, and remove old commented out isNotNull thing (it's no longer relevant)
  • ea3a236 Added a few more tests.
  • 238d362 Reroll isBefore and isAfter the dumb, explicit way.
  • f72ef98 Clean up checkConfig for isBefore and isAfter, and clarify another comment.
  • 167a02d Tweak checkConfig error msgs for isBefore+isAfter
  • ee27777 Add todo for future
  • 0e2805c Fix out of date test label
  • 55d9fbe Clean up tests so they report more helpful stuff.
  • 144d872 Bump validator version re https://snyk.io/test/npm/anchor?severity=high&severity=medium&severity=low#npm:validator:20160218 (without applying any of the necessary changes yet, if there are any). Also upgrade to latest eslintrc file, etc, and bump eslint dep.
  • 41dd3e0 1.2.1
  • 10c7e86 Fix build status urls
  • f6b689b Avoid confusion
  • b429872 More of the same
  • 63fe204 Trivial

See the full diff

Package name: sails-hook-orm The new version differs by 250 commits.
  • 1e9e6f2 2.0.0
  • 58eef54 2.0.0-23
  • 0e2134f Update eslintrc, and fix something trivial
  • 1db6353 Fix https://github.com/Sails request hangs on sails.getDataStore().transaction(fn) , and fn seems not executed when sails-mysql upgraded to 1.0.0-16 balderdashy/sails#4271
  • f2c2fc0 Use `name: 'userError'` instead of sending strings back as error arguments to callbacks.
  • 6f33b31 More friendly error for invalid datastore config
  • 3df2117 Don't allow model instance methods at all.
  • 8b025f1 Update warning msg (now that we're using neDB it needn't come across quite as dire)
  • 6e678df 2.0.0-22
  • 1b5884d Bump parley dep to improve error messages (not to mention all of the other rich, complicated goodness that comes from the latest release)
  • a077af7 Add slightly more accurate suffix to error msg (e.g. to improve the output when attempting to use .transaction() with sails-disk)
  • 1facd90 Properly catch unhandled promise rejections and tolerate non-Errors from procedural parameters (to allow for special exit signals and other flow control paradigms). Solves https://trello.com/c/Yv4BGFwr
  • df71162 Added sanity check w/ better error msg.
  • a1c585d If invalid model definitions are detected, just return an error message _without_ a stack trace.
  • e418590 Validate validation rule configuration
  • d537073 Turn off archiving for tests
  • e21b429 Set default datastore adapter to `sails-disk` (string), and make sure it's loaded last.
  • bc48bd1 trivial
  • 650eec7 Dont globalize models with the _private flag attached.
  • 47d373a trivial
  • 8139c26 2.0.0-21
  • 91580df Bump Waterline version and set to semver range
  • 43c78ed 2.0.0-20
  • 5a96e59 Update scripts and update eslint.

See the full diff

Package name: waterline The new version differs by 250 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


Learn how to fix vulnerabilities with free interactive lessons:

🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

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