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: npm audit reports high vulnerability for handlebars package #367

Closed
yashrajpchavda opened this issue Feb 14, 2019 · 5 comments
Closed

Comments

@yashrajpchavda
Copy link

yashrajpchavda commented Feb 14, 2019

Here is the report of the npm audit

Run npm update handlebars --depth 3 to resolve 1 vulnerability
┌───────────────┬──────────────────────────────────────────────────────────────┐
│ High │ Prototype Pollusion │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Package │ handlebars │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Dependency of │ karma-coverage [dev] │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Path │ karma-coverage > istanbul > handlebars │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ More info │ https://nodesecurity.io/advisories/755

@johnjbarton
Copy link
Contributor

Here are the guides for contributors: http://karma-runner.github.io/3.0/dev/contributing.html

@em0ney
Copy link

em0ney commented Apr 16, 2019

For others who are waiting for this to be sorted out in this module, take a look at this handy module for forcing the resolution of specific versions of transitive dependencies. https://www.npmjs.com/package/npm-force-resolutions

Handy for cases where there is a security vulnerability and the maintainers might take some time to resolve them

@mschaaf
Copy link

mschaaf commented Jun 19, 2019

Is there a chance to get a new release with updated dependencies? What is blocking or missing?

@hicom150
Copy link
Contributor

There is a PR #377 to update the vulnerable package. We can help testing and merging the PR to fix this issue 😉

@scottopherson
Copy link

#382

Looks like a new release would close this issue.

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

7 participants