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 globby security vulnerability #96

Closed
kkmavr opened this issue Nov 4, 2021 · 5 comments · Fixed by #97
Closed

Dependency globby security vulnerability #96

kkmavr opened this issue Nov 4, 2021 · 5 comments · Fixed by #97

Comments

@kkmavr
Copy link

kkmavr commented Nov 4, 2021

There is a vulnerability in the glob-parent library used.
cpy>globby 9.2.0 >fast-glob 2.2.6 > glob-parent 3.1.0

GHSA-ww39-953v-wcq6

Is there a change to update globby to v12.0.0 or a later one, so that it ends up using glop-parent v5.1.2 which is the patched version?

@kkmavr
Copy link
Author

kkmavr commented Nov 8, 2021

Thank you for the fix. Could I ask, when is the next scheduled release, so that we can get this fix?

@sindresorhus
Copy link
Owner

When #92 is merged.

@stof
Copy link

stof commented Nov 17, 2021

@sindresorhus is there any chance to release a non-ESM version of cpy upgrading to globby 10+ (the one using fast-glob 3 and so a patched version of glob-parent), to allow upgrades for downstream packages that cannot migrate to ESM yet ? For instance, for a webpack plugin (I know some of them depending on cpy), it might be hard to migrate to ESM, as I don't think webpack supports ESM config files yet.

@sindresorhus
Copy link
Owner

I don't have any plans to backport. Most users use globby asynchronously and can use the ESM from CommonJS by using await import(…).

@jacob-israel-turner
Copy link

Hey guys! This is blocking security updates for Storybook. Any word on when we'll see #92 merged?

Here are some relevant issues:
storybookjs/storybook#15174
nrwl/nx#5998

Thanks!

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 a pull request may close this issue.

4 participants