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

Adds transitive dependencies (Python) - DO NOT USE FOR INTERVIEWS #9

Closed
wants to merge 1 commit into from

Conversation

pablo-snyk
Copy link
Contributor

@pablo-snyk pablo-snyk commented Dec 18, 2021

In order to extend our vulnerability scanning feature to support child dependencies for npm packages as described in #5 , 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

Unverified

This user has not yet uploaded their public signing key.
@pablo-snyk pablo-snyk changed the title feat: added transitive dependencies Adds transitive dependencies (Python) Dec 18, 2021
@snyk snyk locked as off-topic and limited conversation to collaborators Jan 18, 2022
@rachelcdavies
Copy link

This exercise was revised in 2023, see #15
If you are sent a link to this older PR then please reach out to our talent team to check whether you should prepare a solution for the newer one.

@rachelcdavies
Copy link

rachelcdavies commented Dec 11, 2024

I'm closing this PR to make it even more obvious that no candidates should attempt this old version of the PR exercise.
If you are invited to a PR interview please refer to the open PR #15

@rachelcdavies rachelcdavies changed the title Adds transitive dependencies (Python) Adds transitive dependencies (Python) - DO NOT USE FOR INTERVIEWS Dec 20, 2024
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.

None yet

2 participants