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 due to outdated nested dependency (dot-prop) #259

Open
marian-r opened this issue Jul 29, 2020 · 5 comments
Open

Comments

@marian-r
Copy link

npm audit output:

┌───────────────┬──────────────────────────────────────────────────────────────┐
│ High          │ Prototype Pollution                                          │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Package       │ dot-prop                                                     │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Patched in    │ >=5.1.1                                                      │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Dependency of │ clinic [dev]                                                 │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Path          │ clinic > insight > conf > dot-prop                           │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ More info     │ https://npmjs.com/advisories/1213                            │
└───────────────┴──────────────────────────────────────────────────────────────┘
@mcollina
Copy link
Contributor

There is nothing to worry about this, hopefully it would be fixed soon.

@marian-r
Copy link
Author

@mcollina the vulnerability has been fixed, just a direct dependency (insight) is using an unpatched version of the vulnerable package. Do we just have to wait until "insight" upgrades its dependency?

@mcollina
Copy link
Contributor

that's the plan, yes

@marian-r
Copy link
Author

I've opened an issue for that too sindresorhus/insight#71. Let's hope it won't take long, although I'm a bit fearful it's gonna take a while as that package hasn't seen an update for over a year 😟

@mcollina
Copy link
Contributor

There is literally nothing to worry about that vulnerability, you are safe.

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

2 participants