Skip to content

Obsolete and vulnerable dependencies (CVE-2020-15168 and CVE-2019-20149) #17416

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

Closed
laurentdelosieresmano opened this issue Sep 28, 2020 · 6 comments · Fixed by #19722
Closed
Assignees
Milestone

Comments

@laurentdelosieresmano
Copy link

Bug report

Hello. Several vulnerabilities were revealed in the chokidar and ampproject/toolbox-optimizer package dependencies. The packages causing the "security issues" are kind-of and node-fetch. Detailed info about CVE-2019-20149 and CVE-2020-15168 vulnerabilities are available here ([1] and [2]).

[email protected] > [email protected] > readdirp@^2.2.1 > micromatch@^3.1.10 > braces@^2.3.1 > snapdragon@^0.8.1 > base@^0.11.1 > class-utils@^0.3.5 > define-property@^0.2.5 > is-descriptor@^0.1.0 > is-accessor-descriptor@^0.1.6 > kind-of@^3.0.2

[email protected] > @ampproject/[email protected] > [email protected]

[1] jonschlinkert/kind-of#30
[2] https://nvd.nist.gov/vuln/detail/CVE-2020-15168

@c0b41
Copy link
Contributor

c0b41 commented Sep 28, 2020

This issue already opened, dups #17250

@laurentdelosieresmano
Copy link
Author

Afais, this would address the 2nd vuln but not the first one.

@timneutkens
Copy link
Member

Both reports do not affect Next.js. Feel free to send a PR to upgrade chokidar which should get rid of the mentioned dependency.

@laurentdelosieresmano
Copy link
Author

Hi timneutkens,

Depending on an obsolete/vulnerable dependency is against the security best practices. I will send you a PR to bump the version of chokidar up. Do you have any checks on the pipeline to make sure the new version does not break anything ?

@timneutkens
Copy link
Member

Do you have any checks on the pipeline to make sure the new version does not break anything ?

Yes, integration tests cover all features.

timneutkens added a commit to timneutkens/next.js that referenced this issue Oct 21, 2020
timneutkens added a commit that referenced this issue Oct 21, 2020
@timneutkens timneutkens added this to the iteration 13 milestone Nov 9, 2020
@ijjk ijjk added the point: 1 label Nov 9, 2020
kodiakhq bot pushed a commit that referenced this issue Nov 11, 2020
Updates to the latest version of `chokidar`

x-ref: #17416
@timneutkens timneutkens modified the milestones: iteration 13, iteration 14 Dec 1, 2020
@kodiakhq kodiakhq bot closed this as completed in #19722 Dec 2, 2020
kodiakhq bot pushed a commit that referenced this issue Dec 2, 2020
This updates to the latest version of the `@ampproject/toolbox-optimizer` which contains updates for dependencies to remove the warnings on install from `npm`

x-ref: #18994
Closes: #17416
@balazsorban44
Copy link
Member

This issue has been automatically locked due to no recent activity. If you are running into a similar issue, please create a new issue with the steps to reproduce. Thank you.

@vercel vercel locked as resolved and limited conversation to collaborators Jan 29, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants