Skip to content

[Snyk] Fix for 3 vulnerabilities #43

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

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
⚠️ Warning
Failed to update the package-lock.json, please update manually before merging.

Vulnerabilities that will be fixed

With an upgrade:
Severity Priority Score (*) Issue Breaking Change Exploit Maturity
high severity 589/1000
Why? Has a fix available, CVSS 7.5
Insecure Encryption
SNYK-JS-BCRYPT-572911
Yes No Known Exploit
medium severity 616/1000
Why? Proof of Concept exploit, Has a fix available, CVSS 5.9
Cryptographic Issues
SNYK-JS-BCRYPT-575033
Yes Proof of Concept
medium severity 479/1000
Why? Has a fix available, CVSS 5.3
Insertion of Sensitive Information into Log File
SNYK-JS-NPM-575435
No No Known Exploit

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

Commit messages
Package name: bcrypt The new version differs by 18 commits.
  • 61139e6 v5.0.0
  • 1bde62c Update node-pre-gyp to 0.15.0
  • 40770d6 Add NodeJS 14 to appveyor CI
  • 5916a46 Merge pull request #807 from techhead/known_length
  • f28e916 Reword comment
  • ca1e43b Add test for embedded NULs
  • 1a81858 Pass key_len to bcrypt(). Fix for issues #774, #776
  • cf4efd9 Merge pull request #647 from ilatypov/master
  • 15febd1 Allow using an enterprise artifactory.
  • 96c41e2 Mark z/OS compatibility code as such
  • dd32df1 Add z/OS support
  • ac14738 Update CHANGELOG.md
  • d9e54b4 Merge pull request #806 from techhead/2b_overflow
  • 9548df5 Fix overflow bug. See issue #776
  • 4c38d38 Merge pull request #804 from jokester/add-arm64-build
  • 41d9ba2 add linux-arm64 to build matrix
  • bc114fb Update node-addon-api to v3.0.0
  • 61f6308 Use travis to deploy future releases

See the full diff

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