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

Dependency 'rollup' vulnerability: DOM Clobbering Gadget found in rollup bundled scripts that leads to XSS #3376

Closed
sacru2red opened this issue Nov 19, 2024 · 6 comments · Fixed by #3391
Labels
dependencies Pull requests that update a dependency file

Comments

@sacru2red
Copy link

ref: #3354

"rollup": "^2.43.1",

@westonruter
Copy link
Collaborator

Please elaborate.

@erlichmen
Copy link

I'll try to add:

The version of rollup is quite old and has CVE-2024-47068, all sort of projects that uses workbox are affected by it (we got here by using docusaurus).

github depenbot gives this an 8.3 which is quite high.

@quarryman
Copy link

This is fixed in roll-up 3.29.5, 4.22.4.
Any plans to update roll-up?

@westonruter westonruter added the dependencies Pull requests that update a dependency file label Feb 3, 2025
@westonruter
Copy link
Collaborator

westonruter commented Feb 3, 2025

It seems to also be fixed in v2.79.2, and according to CVE-2024-47068 this version does not have the vulnerability:

Image

This is the version which appears in the main branch:

"rollup": "^2.79.2",

And in the most recent release:

"rollup": "^2.79.2",

This was done in #3359.

@quarryman
Copy link

This has been fixed for workbox directly, workbox-build still points to vulnerable version though

@westonruter westonruter reopened this Feb 4, 2025
@westonruter
Copy link
Collaborator

@quarryman OK, I see that here:

"rollup": "^2.43.1",

And the actual version appearing in package-lock.json is 2.79.1:

"node_modules/rollup": {
"version": "2.79.1",
"resolved": "https://registry.npmjs.org/rollup/-/rollup-2.79.1.tgz",

This needs to be 2.79.2 to include the fix. I'll open a PR to address that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants