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

chore: vulnerability fix - fork and componentize package-json-validator #8850

Merged
merged 6 commits into from
May 2, 2024

Conversation

luvkapur
Copy link
Member

@luvkapur luvkapur commented May 2, 2024

This PR forks the package-json-validator (https://github.com/TechNickAI/package.json-validator/) and componentizes it into @teambit/pkg.package-json.validator component - which is a typescript version of the validator.
The original library had security vulnerability in their cli tool which uses the optimist dependency. This fork removes the cli portion of it (removing our dependency on the vulnerable dependency coming from the original package) and only ports the validator to typescript which we use in legacy to validate package json fields when validating versions.

CR: https://bit.cloud/teambit/toolbox/~change-requests/pkg-package-json-validator

@luvkapur luvkapur requested a review from davidfirst May 2, 2024 20:05
@GiladShoham
Copy link
Member

I would consider moving that component into pkg scope or dependencies scope (probably pkg is better).
And just export it outside and don't make it part of the bit repo at all.

@luvkapur luvkapur enabled auto-merge (squash) May 2, 2024 21:14
@luvkapur luvkapur merged commit c216c11 into master May 2, 2024
11 checks passed
@luvkapur luvkapur deleted the package-json-validator-vulnerability-fix branch May 2, 2024 22:02
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