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

loader-utils security issue #1743

Open
jschuler opened this issue Apr 27, 2023 · 3 comments
Open

loader-utils security issue #1743

jschuler opened this issue Apr 27, 2023 · 3 comments

Comments

@jschuler
Copy link

Hi,

We are trying to remove traces of loader-utils @ v2 from our project OCM, due to a security scan issue https://issues.redhat.com/browse/HAC-2306

Flaw:

CVE-2022-37599 loader-utils: regular expression denial of service in interpolateName.js
https://bugzilla.redhat.com/show_bug.cgi?id=2134872

A Regular expression denial of service (ReDoS) flaw was found in Function interpolateName in interpolateName.js in webpack loader-utils 2.0.0 via the resourcePath variable in interpolateName.js.

loader-utils prior to version 3 is deprecated and no longer supported

External Reference:
https://github.com/webpack/loader-utils/issues/211

https://github.com/RedHatInsights/frontend-components/blob/master/packages/config-utils/chrome-render-loader.js
uses loader-utils getOptions method.

There are 2 issues:

  • There is an implicit dependency on loader-utils since it's not in package.json
  • The implicit dependency is for version 2 of loader-utils, version 3 removed the method in favor of loaderContext

Can this function call be removed?
Thanks!

@Hyperkid123
Copy link
Contributor

@jschuler yeah, this custom webpack loader should no longer be required as we no longer consume custom HTML templates of individual modules.

@jschuler
Copy link
Author

jschuler commented May 1, 2023

@Hyperkid123 ok thank you, sounds like I can remove our usage of that loader then

@Hyperkid123
Copy link
Contributor

Correct.

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

No branches or pull requests

2 participants