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

[SECURITY] CVE in parse-git-config #1482

Open
ajfarkas opened this issue Mar 21, 2025 · 4 comments
Open

[SECURITY] CVE in parse-git-config #1482

ajfarkas opened this issue Mar 21, 2025 · 4 comments

Comments

@ajfarkas
Copy link

Hello! Dependabot alerted me to a prototype pollution vulnerability in parse-git-config. I just want to make sure you're aware of it.

It looks like they are aware of the issue, but do not yet have a fix.

What are your thoughts/plans on mitigating this?

@orta
Copy link
Member

orta commented Mar 21, 2025

No plans, doesn't particularly seem like an exploit you'd be able to use in danger - given that it as already evaluating your on code.

You're welcome to help solve it upstream, then we'll update like other deps

@ajfarkas
Copy link
Author

Reasonable.
We're using it on a FOSS project for government, and they have (for now) strict rules about patching.

@AlbertGazizov
Copy link

You're welcome to help solve it upstream, then we'll update like other deps

Unfortunately, the upstream repo was not updated for 7 years, and the author doesn't respond jonschlinkert/parse-git-config#15

@ashfurrow
Copy link
Member

It seems like the upstream dependency is unlikely to merge the fix. We will need to fork it to resolve the CVE.

@AlbertGazizov do you have any interest in forking? We would need to publish the change to npm as well. Let me know, I can take a crack at it if not 👍

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