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

Vulnerability Tree #1997

Closed
BlythMeister opened this issue Oct 3, 2022 · 5 comments
Closed

Vulnerability Tree #1997

BlythMeister opened this issue Oct 3, 2022 · 5 comments
Labels
enhancement New feature or request p3 Nice-to-have features

Comments

@BlythMeister
Copy link

Current Behavior:

Vulnerability doesn't show the reason your project has that vulnerable component.
You need to work through the project dependencies manually to establish if you can actually take the required update.

Proposed Behavior:

On the vulnerability screen include the upward depenency tree to the top level item.

This way you will be able to identify why you have a reference to the vulnerable dependency.

@BlythMeister BlythMeister added the enhancement New feature or request label Oct 3, 2022
@nscuro
Copy link
Member

nscuro commented Oct 3, 2022

Great suggestion, thanks @BlythMeister!

@nscuro nscuro added the p3 Nice-to-have features label Oct 3, 2022
@rkg-mm
Copy link
Contributor

rkg-mm commented Oct 12, 2022

We are working on big improvements of the dependency-tree, to make it actually useful at all. This would cover stuff of DependencyTrack/frontend#85 and also further ideas.
Basically, we want to make the tree fully navigable, offer buttons in other views next to component name (e.g. vulnerability list, component list,...) which would open the tree expanded to all occurances of that component and probably highlighting them.
We also plan to offer a focus mode if you jumped in like this, which would hide anything not relevant to see the exact upwards path(s). A component could be included multiple times from different paths, which would be well visible then.

I think this would solve your requirement too, just a little different, since I have the same use case

@BlythMeister
Copy link
Author

Yes that does sound great.

At the moment we can see a risk score and what the vulnerabilities are, but once we have that, it's many hours of manual looking for solutions (unless it's in a direct dependency)

@rkg-mm
Copy link
Contributor

rkg-mm commented Oct 12, 2022

Yep, facing the same issue. Workaround until then: I look it up in the originally imported BOM.xml file, there the origination is visible, even though not quickly to look up.

@nscuro nscuro closed this as completed in f376190 Dec 9, 2022
mulder999 pushed a commit to mulder999/dependency-track that referenced this issue Dec 23, 2022
…ncyTrack#2189)

* Add method to return expanded dependency graph

Adds a new API method, which returns a list of components that are
needed to display a dependency graph that is expanded to every
occurrence of a specified component.

Adds new transitive attributes to the component and project class, which
are needed to correctly display and expand the resulting dependency
graph.

Signed-off-by: RBickert <rbt@mm-software.com>

* Return empty list instead of HTTP conflict

Signed-off-by: RBickert <rbt@mm-software.com>

* Return empty list instead of HTTP conflict

Signed-off-by: RBickert <rbt@mm-software.com>

* Fix root components not being found

Signed-off-by: RBickert <rbt@mm-software.com>

* Change UUID in query to parameter

Signed-off-by: RBickert <rbt@mm-software.com>

* Add tests for `getDependencyGraphForComponent`

Change `Component.expand` to `Component.expandDependencyGraph`

Remove string concatenation in `getDependencyGraphForComponent` and
`getParentDependency` in `ComponentQueryManager`

Signed-off-by: RBickert <rbt@mm-software.com>

Signed-off-by: RBickert <rbt@mm-software.com>

Closes DependencyTrack#1997

Signed-off-by: mulder999 <nospam099-github@yahoo.com>
@github-actions
Copy link
Contributor

github-actions bot commented Jan 9, 2023

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 9, 2023
stephan-wolf-ais pushed a commit to AISAutomation/dependency-track that referenced this issue Mar 1, 2023
…ncyTrack#2189)

* Add method to return expanded dependency graph

Adds a new API method, which returns a list of components that are
needed to display a dependency graph that is expanded to every
occurrence of a specified component.

Adds new transitive attributes to the component and project class, which
are needed to correctly display and expand the resulting dependency
graph.

Signed-off-by: RBickert <rbt@mm-software.com>

* Return empty list instead of HTTP conflict

Signed-off-by: RBickert <rbt@mm-software.com>

* Return empty list instead of HTTP conflict

Signed-off-by: RBickert <rbt@mm-software.com>

* Fix root components not being found

Signed-off-by: RBickert <rbt@mm-software.com>

* Change UUID in query to parameter

Signed-off-by: RBickert <rbt@mm-software.com>

* Add tests for `getDependencyGraphForComponent`

Change `Component.expand` to `Component.expandDependencyGraph`

Remove string concatenation in `getDependencyGraphForComponent` and
`getParentDependency` in `ComponentQueryManager`

Signed-off-by: RBickert <rbt@mm-software.com>

Signed-off-by: RBickert <rbt@mm-software.com>

Closes DependencyTrack#1997
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request p3 Nice-to-have features
Projects
None yet
Development

No branches or pull requests

3 participants