[Snyk] Fix for 1 vulnerabilities#751
Conversation
The following vulnerabilities are fixed with an upgrade: - https://snyk.io/vuln/SNYK-JS-JSYAML-13961110
There was a problem hiding this comment.
Pull Request Overview
This PR updates two npm dependencies in the divi-scripts package to address a Prototype Pollution vulnerability (SNYK-JS-JSYAML-13961110) in js-yaml, a transitive dependency.
- Updates css-loader from 0.28.9 to 1.0.0 (major version bump)
- Updates grunt from 1.0.2 to 1.2.0 (minor version bump)
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "case-sensitive-paths-webpack-plugin": "2.1.1", | ||
| "chalk": "2.3.0", | ||
| "css-loader": "0.28.9", | ||
| "css-loader": "1.0.0", |
There was a problem hiding this comment.
The upgrade to css-loader 1.0.0 introduces a breaking change. The minimize option was removed in css-loader 1.0.0 and moved to optimization configuration. The webpack configuration files (webpack.config.prod.js) currently use minimize: true in the css-loader options (lines 323, 362, 397), which will cause build failures.
To fix this, the minimize option should be removed from css-loader options and handled through webpack's optimization configuration instead, or the webpack config files should be updated accordingly.
Snyk has created this PR to fix 1 vulnerabilities in the npm dependencies of this project.
Snyk changed the following file(s):
packages/divi-scripts/package.jsonVulnerabilities that will be fixed with an upgrade:
SNYK-JS-JSYAML-13961110
Important
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
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic
Learn how to fix vulnerabilities with free interactive lessons:
🦉 Prototype Pollution