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

Remediation of CVE-2023-2251 #2002

Merged
merged 2 commits into from
Jun 16, 2023
Merged

Conversation

yeenbean
Copy link
Contributor

A vulnerability in npm package yaml was discovered in April this year. Details about this vulnerability can be reviewed here: GHSA-f9xv-q969-pqx4.

npm package lint-staged currently targets yaml version 2.2.1, which is an effected version of the package. This pull request resolves this issue by creating a package resolution for lint-staged, targeting version 2.2.2 and greater.

@yeenbean yeenbean marked this pull request as ready for review June 16, 2023 06:45
@@ -56,5 +56,8 @@
"lint-staged": "^13.2.0",

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Might be something to consider. My objective was the path of least resistance but this is probably the way.

Copy link

@ThatOneCalculator ThatOneCalculator left a comment

Choose a reason for hiding this comment

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

yarn.lock will also have to be updated (by running yarn install).

@yeenbean
Copy link
Contributor Author

yarn.lock will also have to be updated (by running yarn install).

Thanks for catching that @ThatOneCalculator. Still a yarn noob 🙃

@ThatOneCalculator
Copy link

No worries! You should run yarn install again though after pinning lint-staged ^^;

@Yidadaa Yidadaa merged commit 7ebc5eb into ChatGPTNextWeb:main Jun 16, 2023
alchemist139 pushed a commit to alchemist139/ChatGPT-Next-Web that referenced this pull request Sep 21, 2023
gaogao1030 pushed a commit to gaogao1030/ChatGPT-Next-Web that referenced this pull request May 16, 2024
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

Successfully merging this pull request may close these issues.

3 participants