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

Add new endpoint for DependencyGraph #366

Merged
merged 1 commit into from
Oct 20, 2023

Conversation

sahibamittal
Copy link
Collaborator

@sahibamittal sahibamittal commented Oct 19, 2023

Description

In this PR I create two new endpoints GET /api/v1/dependencyGraph/project/{uuid}/directDependencies and
GET /api/v1/dependencyGraph/component/{uuid}/directDependencies.

The purpose of this endpoint it's to avoid the frontend from doing multiple requests to the backend to fetch each node of the Dependency Graph. With this endpoint, you can get all directDependencies of a project or a component by providing his UUID.

Ports DependencyTrack/dependency-track#3083 and DependencyTrack/dependency-track#2623.

Addressed Issue

DependencyTrack/hyades#858

Checklist

  • I have read and understand the contributing guidelines
  • This PR fixes a defect, and I have provided tests to verify that the fix is effective
  • This PR implements an enhancement, and I have provided tests to verify that it works as intended
  • This PR introduces changes to the database model, and I have added corresponding update logic
  • This PR introduces new or alters existing behavior, and I have updated the documentation accordingly

@nscuro
Copy link
Member

nscuro commented Oct 19, 2023

@sahibamittal Please use Co-authored-by to denote the original author of the change. I don't care about changes that I did, but for other contributors we should give credit where credit is due. See https://docs.github.com/en/pull-requests/committing-changes-to-your-project/creating-and-editing-commits/creating-a-commit-with-multiple-authors#creating-co-authored-commits-on-the-command-line

Co-Authored-By: Niklas <[email protected]>
Co-Authored-By: Nathan Mittelette <[email protected]>
@sahibamittal sahibamittal force-pushed the backport-endpoint-dependency-graph branch from c7f1f0a to 46f9804 Compare October 19, 2023 13:58
@nscuro nscuro added the enhancement New feature or request label Oct 20, 2023
@nscuro nscuro merged commit a1c41cb into main Oct 20, 2023
6 checks passed
@nscuro nscuro deleted the backport-endpoint-dependency-graph branch October 20, 2023 11:20
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants