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

Security vulnerability in mem dependency #561

Closed
Harmon758 opened this issue Jul 15, 2019 · 2 comments
Closed

Security vulnerability in mem dependency #561

Harmon758 opened this issue Jul 15, 2019 · 2 comments
Labels
dependencies Pull requests that update a dependency file security

Comments

@Harmon758
Copy link

The version of yargs being used has a mem dependency with a security vulnerability:
sindresorhus/memoize#14 (Fixed with sindresorhus/memoize@da4e439 / sindresorhus/memoize#19)
sindresorhus/os-locale#31
yargs/yargs#1209 (Fixed with yargs>=12.0.2: yargs/yargs@efc0970 / yargs/yargs#1195)

Dependency hierarchy:

`-- idyll@4.5.3
  `-- yargs@8.0.1
    `-- os-locale@2.1.0
      `-- mem@1.1.0

"yargs": "8.0.1"

GitHub has recently started creating security alerts for this vulnerability:
image

@mathisonian
Copy link
Member

mathisonian commented Jul 16, 2019

I haven't seen a security alert for this yet - we released a dependency update a few days ago (#558) is it possible this is already fixed in the latest?

edit: never mind, just checked yarn.lock and confirmed mem@1.1.0 is still there.

@mathisonian mathisonian added security dependencies Pull requests that update a dependency file labels Jul 16, 2019
@Harmon758
Copy link
Author

Resolved by 8610555 (#562) in idyll@4.6.0.

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 security
Projects
None yet
Development

No branches or pull requests

2 participants