Skip to content

feat: resolve full dependency tree #12

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nicarl
Copy link
Contributor

@nicarl nicarl commented Mar 20, 2025

In order to extend our vulnerability scanning feature to support child dependencies for npm packages as described in #4 , this pull request updates the package service to return all nested dependencies on the internal /package endpoint for consumption by vulnerability service instead of just returning the versions for the first level.

It supports a GET request to the /package/:name/:version endpoint and will return a JSON structure representing the full tree of dependencies. We will always return the latest matching version of a package supported to mimic the behavior of a fresh npm install.

Testing

It can be tested locally by making a request for a package with sub-dependencies e.g. [email protected]

curl -s http://localhost:3000/package/react/16.13.0 | jq .

Related Ticket

@snyk snyk deleted a comment from orabrenergottlieb Apr 8, 2025
@snyk snyk deleted a comment from orabrenergottlieb Apr 8, 2025
@snyk snyk locked and limited conversation to collaborators Apr 8, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant