-
Notifications
You must be signed in to change notification settings - Fork 28
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 1 vulnerabilities #527
Conversation
Codecov Report
@@ Coverage Diff @@
## master #527 +/- ##
=========================================
Coverage 56.07% 56.07%
Complexity 263 263
=========================================
Files 17 17
Lines 922 922
=========================================
Hits 517 517
Misses 405 405 Continue to review full report at Codecov.
|
@micbar @phil-davis wouldn't it be good to fix a vulnerable package before 10.4 gets out? |
@micbar I have no idea about assessing |
Pls also see #523 |
@IljaN Please review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 💃
In case of merging, is there a changelog entry necessary? |
4565f42
to
ee5329b
Compare
The following vulnerabilities are fixed with an upgrade: - https://snyk.io/vuln/SNYK-JS-DOTPROP-543489
ee5329b
to
4bf3bad
Compare
Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.
Changes included in this PR
Vulnerabilities that will be fixed
With an upgrade:
SNYK-JS-DOTPROP-543489
Commit messages
Package name: snyk
The new version differs by 58 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