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

Bumbing http-cache-semantics to 4.1.1 to fix cve #832

Merged
merged 1 commit into from
Oct 9, 2024

Conversation

gmatuz
Copy link

@gmatuz gmatuz commented Oct 9, 2024

Fixes DoS issues from http-cache-semantics dependency version 4.1.0 by bumping to 4.1.1
GHSA-rc47-6667-2j5j

Most of the dependencies are quite outdated but currently runs under node 13 with npm ci I tried to change as little as necessary

Does not work on more recent node version since py-gyp won't build probaby because the py-gyp is not maintained anymore

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Oct 9, 2024
Copy link
Contributor

@ChrisCummins ChrisCummins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you for the contribution @gmatuz!

Cheers,
Chris

@ChrisCummins ChrisCummins merged commit 00cd50f into facebookresearch:development Oct 9, 2024
17 of 25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants