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

High vulnerability in 'braces' dependency #254

Closed
apptio-mrejdych opened this issue May 15, 2024 · 5 comments
Closed

High vulnerability in 'braces' dependency #254

apptio-mrejdych opened this issue May 15, 2024 · 5 comments

Comments

@apptio-mrejdych
Copy link

Hey Team
Snyk found high vulnerability in your package connected to 'braces' dependency

https://security.snyk.io/vuln/SNYK-JS-BRACES-6838727
https://www.cve.org/CVERecord?id=CVE-2024-4068

Is there any chance to fix it?

@gagadzq
Copy link

gagadzq commented May 15, 2024

I encountered the same problem. Maybe Braces team is working on resolving this issue. Refer to micromatch/braces#36

@paulmillr
Copy link
Member

  1. It is not a real vulnerability. It is another shit that got cve [BUG] Vulnerabilities Found in Micromatch and Braces #243
  2. CVE rating 7.5 is nonsense. More like 2.5
  3. They can’t even produce a working ezploit
  4. There are no other packages to switch. They are either esm only, or very slow, or potentially dangerous with unknown maintainers

@Experimental-products
Copy link

If anyone is hoping to release software without any knownvulnerabilities this is a problem, however we judge the severity ourselves. The problem exists, colleagues/coworkers/clients/customers of us all will likely be put off by this issue as more and more security tools flag this issue.

As the screenshot below indicates, "braces" has not had any updates in 5 years, so maybe micromatch can look at switching to an alternative that is better maintained?

image

@paulmillr
Copy link
Member

micromatch is made by braces founders...

@paulmillr
Copy link
Member

anyways will close it since braces fix is in and we just need to release it. micromatch will autofetch the update.

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

4 participants