kelektiv / node.bcrypt.js

bcrypt for NodeJs
MIT License
7.43k stars 510 forks source link

[Snyk] Security upgrade @mapbox/node-pre-gyp from 1.0.2 to 1.0.6 #931

Closed recrsn closed 1 year ago

recrsn commented 2 years ago

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 - package-lock.json #### Vulnerabilities that will be fixed ##### With an upgrade: Severity | Priority Score (*) | Issue | Breaking Change | Exploit Maturity :-------------------------:|-------------------------|:-------------------------|:-------------------------|:------------------------- ![high severity](https://res.cloudinary.com/snyk/image/upload/w_20,h_20/v1561977819/icon/h.png "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](https://snyk.io/vuln/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: @mapbox/node-pre-gyp The new version differs by 45 commits.
  • 4f5b513 bump to v1.0.6
  • 233785e remove debug prints [skip ci]
  • 9572c26 attempt to fix node less than 16 which contains older node-gyp before 0.6.0 release - refs nodejs/node-gyp#2233
  • 6356cc7 start assuming newly cross-platform shared_library location in node-gyp
  • fa143c9 attempt to fix app4 test
  • 9abbf6f more node v16 image debugging
  • 8f9e8ad Merge pull request #615 from mapbox/dependabot/npm_and_yarn/path-parse-1.0.7
  • 6ea0f83 debug node v16 test failures
  • d3249e0 fix missing test target
  • 1843a70 Bump path-parse from 1.0.6 to 1.0.7
  • 2816131 try bionic to fix python issue with node-gyp
  • 77cdd6e ensure python is installed on travis (fixes node v16 build)
  • 7077464 add node v16 / disable node-webkit testing on CI - #614
  • 2441352 downgrade nock to attempt to fix 'Cannot read property isMap'
  • a59f5eb start changelog for v1.0.6
  • d4116cd Merge branch 'master' of github.com:mapbox/node-pre-gyp
  • a5c4880 use eslint@7.x to be able to continue to test with node v10
  • 7f363ca Merge pull request #590 from ronilan/Fix-Unpublish-Default-Issue-589
  • 3ac4191 Merge pull request #599 from codeplea/master
  • 3aadedf upgrade deps
  • 1c13991 Merge pull request #601 from mapbox/dependabot/npm_and_yarn/tar-6.1.2
  • 039213c Merge pull request #602 from mapbox/dependabot/npm_and_yarn/lodash-4.17.21
  • 6c720fc Merge pull request #603 from mapbox/dependabot/npm_and_yarn/glob-parent-5.1.2
  • 81bcaa6 Merge pull request #612 from ewanharris/add_node_17
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](https://app.snyk.io/org/agathver/project/e23f49f4-716e-4b09-ad65-2065c99d1f0f?utm_source=github&utm_medium=referral&page=fix-pr) 🛠 [Adjust project settings](https://app.snyk.io/org/agathver/project/e23f49f4-716e-4b09-ad65-2065c99d1f0f?utm_source=github&utm_medium=referral&page=fix-pr/settings) 📚 [Read more about Snyk's upgrade and patch logic](https://support.snyk.io/hc/en-us/articles/360003891078-Snyk-patches-to-fix-vulnerabilities) [//]: # (snyk:metadata:{"prId":"398b5aeb-c338-4190-89c3-570bc2d2f75c","prPublicId":"398b5aeb-c338-4190-89c3-570bc2d2f75c","dependencies":[{"name":"@mapbox/node-pre-gyp","from":"1.0.2","to":"1.0.6"}],"packageManager":"npm","projectPublicId":"e23f49f4-716e-4b09-ad65-2065c99d1f0f","projectUrl":"https://app.snyk.io/org/agathver/project/e23f49f4-716e-4b09-ad65-2065c99d1f0f?utm_source=github&utm_medium=referral&page=fix-pr","type":"auto","patch":[],"vulns":["SNYK-JS-ANSIREGEX-1583908"],"upgrade":["SNYK-JS-ANSIREGEX-1583908"],"isBreakingChange":false,"env":"prod","prType":"fix","templateVariants":["updated-fix-title","priorityScore","merge-advice-badge-shown"],"priorityScoreList":[696]}) --- **Learn how to fix vulnerabilities with free interactive lessons:** 🦉 [Learn about vulnerability in an interactive lesson of Snyk Learn.](https://learn.snyk.io?loc=fix-pr)
ghost commented 2 years ago

Can this be merged in? Seems like it's only failing on node 10 and 11 which is years out of date.

recrsn commented 1 year ago

closed as merged with another pr