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

New vulnerability CVE-2022-32210 found on v18.x #45

Closed
github-actions bot opened this issue Aug 3, 2022 · 4 comments
Closed

New vulnerability CVE-2022-32210 found on v18.x #45

github-actions bot opened this issue Aug 3, 2022 · 4 comments

Comments

@github-actions
Copy link

github-actions bot commented Aug 3, 2022

Failed run: https://github.com/nodejs/nodejs-dependency-vuln-assessments/actions/runs/2790721148
Vulnerability ID: CVE-2022-32210

Full output:

WARNING: New vulnerabilities found
- undici (version 5.8.0) :
	- CVE-2022-31151: https://nvd.nist.gov/vuln/detail/CVE-2022-31151
	- CVE-2022-31150: https://nvd.nist.gov/vuln/detail/CVE-2022-31150
	- CVE-2022-32210: https://nvd.nist.gov/vuln/detail/CVE-2022-32210

For each dependency and vulnerability, check the following:
- Check that the dependency's version printed by the script corresponds to the version present in the Node repo.
If not, update dependencies.py with the actual version number and run the script again.
- If the version is correct, check the vulnerability's description to see if it applies to the dependency as
used by Node. If not, the vulnerability ID (either a CVE or a GHSA) can be added to the ignore list in
dependencies.py. IMPORTANT: Only do this if certain that the vulnerability found is a false positive.
- Otherwise, the vulnerability found must be remediated by updating the dependency in the Node repo to a
non-affected version, followed by updating dependencies.py with the new version.
@mhdawson
Copy link
Member

mhdawson commented Aug 3, 2022

Check - #10 for confirmation that this does not affect Node.js

@mhdawson
Copy link
Member

mhdawson commented Aug 3, 2022

@facutuesca in respect to -#10 (comment) could you take another look to see if it still makes sense that its beeing reported as a vulnerability.

@facutuesca
Copy link
Contributor

@mhdawson I found the issue: It is still being reported as a vulnerability even after the NVD updated the affected versions, due to the kind of query the script does to search for undici in the database.

I opened a small PR with the fix here: nodejs/node#44128

@mhdawson
Copy link
Member

mhdawson commented Aug 4, 2022

@facutuesca thanks, approved and requested fast track to see if we can get it landed soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants