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 2 vulnerabilities #7

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

Conversation

iketheadore
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.

Changes included in this PR

  • Changes to the following files to upgrade the vulnerable dependencies to a fixed version:
    • js/package.json
    • js/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
high severity 589/1000
Why? Has a fix available, CVSS 7.5
Prototype Pollution
SNYK-JS-UNSETVALUE-2400660
No No Known Exploit

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

Commit messages
Package name: node-test The new version differs by 49 commits.
  • bf65670 1.4.2
  • 318d9cb updated for changes to async hooks api
  • cff5436 1.4.1
  • 4ad5f55 minor fixes
  • 7b8a6c7 1.4.0
  • 577f356 all tests passing
  • 49b0b6f fixes for hung context
  • 2c22b25 long stack traces test passing
  • e8ccae5 concurrent fail fast working
  • 06b31ce fail fail serial working
  • bb2d4d4 test hooks working
  • 17eed6a before and after hooks working
  • 7f71d7e fixes for contexts
  • 4e0b0ec Contexts working with traces
  • b8d4b0a separated traces
  • a63f4f3 added back test timeouts
  • b6bbffc overload more Error types
  • ce89b41 Context fixes
  • ca8b0d9 fixed case require bug
  • 086aa43 create Context object that extends AsyncResource and provide context functionality
  • c3eb1a2 only store relationship between async resource if there is a context of the parent
  • 116e76a split stack trace from context
  • 1c6591c fixes for caught exceptions
  • 355fe0f uncaught exceptions routed to correct error handler

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)
🦉 Prototype Pollution

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

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