Skip to content
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

react-scripts is using postcss@^7.0.35 which has security vulnerability #13423

Open
biaoqiu opened this issue Oct 30, 2023 · 4 comments · May be fixed by #13454
Open

react-scripts is using postcss@^7.0.35 which has security vulnerability #13423

biaoqiu opened this issue Oct 30, 2023 · 4 comments · May be fixed by #13454

Comments

@biaoqiu
Copy link

biaoqiu commented Oct 30, 2023

react-scripts@5.0.1 requires postcss@^7.0.35 via a transitive dependency on resolve-url-loader@4.0.0

I see the latest version of resolve-url-loader is 5.x, and it depends on postcss@8.x. So can we update resolve-url-loader to a non-vulnerable version? Thank you!

@JcPires
Copy link

JcPires commented Nov 2, 2023

A link to the CVE: https://nvd.nist.gov/vuln/detail/CVE-2023-44270

vavsab added a commit to vavsab/create-react-app that referenced this issue Nov 17, 2023
@vavsab vavsab linked a pull request Nov 17, 2023 that will close this issue
@LiveDream0630
Copy link

react-scripts@5.0.1 requires postcss@^7.0.35 via a transitive dependency on resolve-url-loader@4.0.0

I see the latest version of resolve-url-loader is 5.x, and it depends on postcss@8.x. So can we update resolve-url-loader to a non-vulnerable version? Thank you!

Actually, in our project, we don't import the resolve-url-loader package directly. Only react-scripts module has some dependencies with resolve-url-loader and here I get vulnerabilities.
So in this case, how can I address them?
Do I have to change the version of react-scripts or just need to install proper version of resolve-url-loader separately?

@Dror-Bar
Copy link

I had to use overrides in my package.json to overcome there errors:

  "overrides": {
    "nth-check": "https://registry.npmjs.org/nth-check/-/nth-check-2.1.1.tgz",
    "postcss": "https://registry.npmjs.org/postcss/-/postcss-8.4.31.tgz"
  }

However the dependency should be updated in the main branch

@AnaisUrlichs
Copy link

@Dror-Bar thank you, you are going to be in my video on using Trivy to fix vulnerabilities with this suggestion -- Thank you!!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants