Skip to content

chore: security update for terser-webpack-plugin-legacy#330

Merged
alexander-akait merged 2 commits intowebpack:webpack-3from
antonku:webpack-3
Oct 8, 2020
Merged

chore: security update for terser-webpack-plugin-legacy#330
alexander-akait merged 2 commits intowebpack:webpack-3from
antonku:webpack-3

Conversation

@antonku
Copy link
Copy Markdown

@antonku antonku commented Oct 8, 2020

This PR contains a:

  • bugfix
  • new feature
  • code refactor
  • test update
  • typo fix
  • metadata update

Motivation / Use-Case

Hi. I assume that terser-webpack-plugin-legacy is unsupported taking into account that the notice regarding it is no longer in the readme. Still, is there any chance that you could publish a security update for it? Thanks.

This PR fixes: CVE-2019-16769 and CVE-2020-7660

@jsf-clabot
Copy link
Copy Markdown

jsf-clabot commented Oct 8, 2020

CLA assistant check
All committers have signed the CLA.

@alexander-akait alexander-akait merged commit 5ed0d20 into webpack:webpack-3 Oct 8, 2020
@alexander-akait
Copy link
Copy Markdown
Member

Released (1.2.4), my strong recommendation to migrate on webpack 4 or webpack 5 (recommended)

@antonku
Copy link
Copy Markdown
Author

antonku commented Oct 8, 2020

@evilebottnawi Thank you so much, you saved the day!

One more thing, I have noticed that in 1.2.4 there are several core-js require statements in dist/index.js and dist/minify.js, specifically:

require("core-js/modules/es7.object.get-own-property-descriptors");

require("core-js/modules/es6.object.to-string");

These imports may be a breaking change for consumers that don't have core-js@2.
Is it an intended change?

@alexander-akait
Copy link
Copy Markdown
Member

Yes, we need it because we support old Node.js versions, but maybe we can improve it, feel free to send a PR

@antonku
Copy link
Copy Markdown
Author

antonku commented Oct 8, 2020

Yes, we need it because we support old Node.js versions, but maybe we can improve it, feel free to send a PR

Got it, thanks. I think it makes sense to add core-js@2 to dependencies of terser-webpack-plugin-legacy so that end users without core-js@2 wouldn't experience issues during terser-webpack-plugin-legacy update.

I have opened a PR for it, please take a look: #331

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants