Skip to content
This repository has been archived by the owner on Dec 2, 2022. It is now read-only.

Struct fix #18

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Struct fix #18

wants to merge 2 commits into from

Conversation

l0cs
Copy link

@l0cs l0cs commented Apr 21, 2022

I added some missing fields from the Repository endpoint and included them in the Component struct to save having to make 2 calls.

@nscuro
Copy link
Owner

nscuro commented Apr 29, 2022

Hi @l0cs 👋

Thanks for the PR! But is the repositoryMeta field actually returned by the API? I just tested with a local DT instance, and the field is not present in responses to the /api/v1/component/<UUID> request.

The field is also transient and thus should not be serialized at all: https://github.com/DependencyTrack/dependency-track/blob/master/src/main/java/org/dependencytrack/model/Component.java#L330

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.

2 participants