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

npm 8.19.3: security issue in the http-cache-semantics #6151

Closed
mihob opened this issue Feb 9, 2023 · 7 comments
Closed

npm 8.19.3: security issue in the http-cache-semantics #6151

mihob opened this issue Feb 9, 2023 · 7 comments

Comments

@mihob
Copy link

mihob commented Feb 9, 2023

There is a security issue in the http-cache-semantics package used by the make-fetch-happen package.

The current version of make-fetch-happen uses a version of http-cache-semantics in which the problem is fixed.

Would it be possible to update the dependencies accordingly?

@Morl99
Copy link

Morl99 commented Feb 9, 2023

That should be fixed with v9.4.2

@mihob
Copy link
Author

mihob commented Feb 9, 2023

It may be fixed for 9.x, but not for 8.x

@ljharb
Copy link
Contributor

ljharb commented Feb 9, 2023

@mihob 8.x is EOL and won't likely ever be fixed. only npm 6 and npm 9 are still getting updates afaik.

@denholtz
Copy link

denholtz commented Feb 9, 2023

This is just waiting on a merge.
https://github.com/npm/cli/pull/6148/files

@Morl99
Copy link

Morl99 commented Feb 9, 2023

Is there a specific reason to not update to npm 9?

@wraithgar
Copy link
Member

regardless of how this gets backported to npm 8, npm 8 itself is not vulnerable. This is only causing an npm audit warning because the module itself has a vulnerability at that version. The vulnerability is when http-cache-semantics is used to parse requests, which the npm cli does not do.

@okuryu
Copy link

okuryu commented Feb 16, 2023

It looks like v8.19.4 has http-cache-semantics update.
https://github.com/npm/cli/releases/tag/v8.19.4

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