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

Upgrade appmetrics to latest version #235

Open
hsandovalskytap opened this issue Apr 15, 2019 · 3 comments
Open

Upgrade appmetrics to latest version #235

hsandovalskytap opened this issue Apr 15, 2019 · 3 comments

Comments

@hsandovalskytap
Copy link

Hello,

recently we noticed there is a security vulnerability in node-tar which is being used by one of your dependencies, appmetrics

└─┬ [email protected] └─┬ [email protected] └── [email protected]

appmetrics has already addressed the problem but you would need to upgrade to the latest version, [email protected].

Could you help us out upgrading appmetrics to the latest version?

@sam-github
Copy link
Contributor

I'm not sure who is maintaining this anymore. That said:

  1. test don't pass with appmetrics@4. This could be related to appmetrics, or not, and it doesn't have a changelog (that I found).
  2. the vulnerabilities in node-tar are irrelevant to appmetrics. The vulnnerability affects using node-tar to untar user-provided tarballs (which could have poison data). Appmetrics uses tar only sometimes, and always to untar tarballs that are part of its distribution.

@scottbrady
Copy link

Hello! Is there an ETA to remediate the advisory on this package? npm/yarn audit is showing a vulnerability for this package's dependency.

┌───────────────┬──────────────────────────────────────────────────────────────┐
│ high          │ Arbitrary File Overwrite                                     │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Package       │ tar                                                          │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Patched in    │ >=4.4.2                                                      │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Dependency of │ strong-supervisor                                            │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Path          │ strong-supervisor > appmetrics > tar                         │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ More info     │ https://www.npmjs.com/advisories/803                         │
└───────────────┴──────────────────────────────────────────────────────────────┘
┌───────────────┬──────────────────────────────────────────────────────────────┐
│ high          │ Arbitrary File Overwrite                                     │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Package       │ tar                                                          │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Patched in    │ >=4.4.2                                                      │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Dependency of │ strong-supervisor                                            │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Path          │ strong-supervisor > appmetrics-dash > appmetrics > tar       │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ More info     │ https://www.npmjs.com/advisories/803                         │
└───────────────┴──────────────────────────────────────────────────────────────┘

Thanks!

@smartmouse
Copy link

yes, can we upped to appmetrics dependencies to latest version which is 5.x?
Got into some issues with installing the [email protected] on windows.

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

No branches or pull requests

4 participants