[Snyk] Security upgrade node-sass from 6.0.1 to 9.0.0#27
[Snyk] Security upgrade node-sass from 6.0.1 to 9.0.0#27karimkawambwa wants to merge 1 commit intomasterfrom
Conversation
The following vulnerabilities are fixed with an upgrade: - https://snyk.io/vuln/SNYK-JS-LODASH-15053838
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| "css-loader": "^6.2.0", | ||
| "mini-css-extract-plugin": "^2.1.0", | ||
| "node-sass": "^6.0.1", | ||
| "node-sass": "^9.0.0", |
There was a problem hiding this comment.
Bug: The yarn.lock file was not updated, so the old, vulnerable version of node-sass will be installed, and the security fix will not be applied.
Severity: MEDIUM
Suggested Fix
Run yarn install or yarn upgrade node-sass locally to regenerate the yarn.lock file with the correct dependency version (^9.0.0) and commit the updated file. This will ensure the intended version is installed during builds.
Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent.
Verify if this is a real issue. If it is, propose a fix; if not, explain why it's not
valid.
Location: package.json#L17
Potential issue: The `package.json` file was updated to use `node-sass` version `^9.0.0`
to address a security vulnerability. However, the `yarn.lock` file was not updated and
still specifies version `^6.0.1`. When `yarn install` is run, it will prioritize the
lockfile and install the older, vulnerable version of `node-sass`. As a result, the
intended security fix will not be applied, and the project will remain exposed to the
vulnerability the pull request was meant to resolve.
Did we get this right? 👍 / 👎 to inform future reviews.
Snyk has created this PR to fix 1 vulnerabilities in the yarn dependencies of this project.
Snyk changed the following file(s):
package.jsonNote for zero-installs users
If you are using the Yarn feature zero-installs that was introduced in Yarn V2, note that this PR does not update the
.yarn/cache/directory meaning this code cannot be pulled and immediately developed on as one would expect for a zero-install project - you will need to runyarnto update the contents of the./yarn/cachedirectory.If you are not using zero-install you can ignore this as your flow should likely be unchanged.
Vulnerabilities that will be fixed with an upgrade:
SNYK-JS-LODASH-15053838
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