ExtensionEngine / tailor

Content authoring platform
MIT License
31 stars 10 forks source link

[Snyk] Security upgrade read-pkg-up from 7.0.1 to 8.0.0 #802

Closed snyk-bot closed 3 years ago

snyk-bot commented 3 years ago

Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.

merge advice

Changes included in this PR

Vulnerabilities that will be fixed

With an upgrade:
Severity Priority Score (*) Issue Breaking Change Exploit Maturity
medium severity 658/1000
Why? Proof of Concept exploit, Recently disclosed, Has a fix available, CVSS 5.3
Regular Expression Denial of Service (ReDoS)
SNYK-JS-HOSTEDGITINFO-1088355
Yes Proof of Concept

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

Commit messages
Package name: read-pkg-up The new version differs by 3 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

ikovac commented 3 years ago

Version 8 of the read-pkg-up is using ESM module system. According to https://gist.github.com/sindresorhus/a39789f98801d908bbc7ff3ecc99d99c we should stay on version 7 because ESM modules cannot be imported using commonJS.

vladimyr commented 3 years ago

I guess at some point we'll be simply forced to move to ESM as number of deps jumping on this crazy bandwagon increases.