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

[GHSA-7q7g-4xm8-89cq] Regular Expression Denial of Service (ReDoS) in @eslint/plugin-kit #5014

Conversation

MikuroXina
Copy link

Updates

  • CVSS v3
  • Severity

Comments
It lacks the reason why the severity is high. According to this description, I think that the CVSS v3 vector string is CVSS:3.1/AV:N/AC:L/PR:L/UI:R/S:U/C:N/I:N/A:L and the assessment results low.

@github
Copy link
Collaborator

github commented Nov 16, 2024

Hi there @mdjermanovic! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.

This change will be reviewed by our Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory

@github-actions github-actions bot changed the base branch from main to MikuroXina/advisory-improvement-5014 November 16, 2024 14:12
Copy link

@nzakas nzakas left a comment

Choose a reason for hiding this comment

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

I agree. I think this was misclassified as high.

@darakian
Copy link
Contributor

Hey, so we pulled in the severity from the repo advisory here
GHSA-7q7g-4xm8-89cq
@nzakas it looks like you're a maintainer on that repo. Would you mind updating that advisory to what you think a correct CVSS score should be and we can reflect that.

@nzakas
Copy link

nzakas commented Nov 18, 2024

@darakian done.

@darakian
Copy link
Contributor

darakian commented Nov 18, 2024

Cool. I'll go ahead and use CVSS:3.1/AV:N/AC:L/PR:L/UI:R/S:U/C:N/I:N/A:L for the string then as suggested by @MikuroXina then 👍

@advisory-database advisory-database bot merged commit 393c868 into MikuroXina/advisory-improvement-5014 Nov 18, 2024
2 checks passed
@advisory-database
Copy link
Contributor

Hi @MikuroXina! Thank you so much for contributing to the GitHub Advisory Database. This database is free, open, and accessible to all, and it's people like you who make it great. Thanks for choosing to help others. We hope you send in more contributions in the future!

@advisory-database advisory-database bot deleted the MikuroXina-GHSA-7q7g-4xm8-89cq branch November 18, 2024 20:49
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.

4 participants