Skip to content

feat: Add TagService #45

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 10 commits into from
May 20, 2025
Merged

feat: Add TagService #45

merged 10 commits into from
May 20, 2025

Conversation

@SolarFactories
Copy link

@SolarFactories SolarFactories commented May 17, 2025

  • Add TagService to be able to create, and delete tags.
  • Provide bulk operations to be able tag and untag projects, policies and notification rules.
  • Add tests for bulk operations of projects, policies.
    • Not added for notification rules, as they are not managed within SDK at present.

@SolarFactories
Added TagService, with Create, GetAll, Delete. Added SortOptions and …
23d3592 
…corresponding withSortOptions.

Signed-off-by: SolarFactories <[email protected]>
@SolarFactories
Added tests for Tag operations.
98138d4 
Signed-off-by: SolarFactories <[email protected]>
@SolarFactories
Added tests for count attributes within TagListResponseItem.
026805f 
Signed-off-by: SolarFactories <[email protected]>
@SolarFactories
Added method to bulk tag projects.
a5a04ee 
Signed-off-by: SolarFactories <[email protected]>
@SolarFactories
Added remainder of project, policy, notificationRule methods to TagSe…
10d84ba 
…rvice.

Signed-off-by: SolarFactories <[email protected]>
@SolarFactories
Added tests for bulk project operations on tags.
f625189 
Signed-off-by: SolarFactories <[email protected]>
@SolarFactories
Added tests for TagService bulk tag operations on policies.
cdd5e95 
Signed-off-by: SolarFactories <[email protected]>
@SolarFactories
Fixed test for TestTag to expect int64, rather than int.
2436c6a 
Signed-off-by: SolarFactories <[email protected]>
@SolarFactories SolarFactories requested a review from nscuro as a code owner May 17, 2025 21:36
@owasp-dt-bot
Copy link

owasp-dt-bot commented May 17, 2025 •
edited
Loading

🎉 Snyk checks have passed. No issues have been found so far.

✅ security/snyk check is complete. No issues have been found. (View Details)

nscuro
nscuro requested changes May 19, 2025
View reviewed changes
@SolarFactories
Added API version asserts for functions within TagService.
5cdb1c7 
Signed-off-by: SolarFactories <[email protected]>
@SolarFactories
Added TagService.GetTagsForPolicy, switching API method based on vers…
57c4f4b 
…ion semver due to deprecation. Moved checks within TestTagPolicies to their scoped blocks, as they do not introduce any required variables after the initial setup.

Signed-off-by: SolarFactories <[email protected]>
@SolarFactories SolarFactories requested a review from nscuro May 19, 2025 21:56
@nscuro nscuro added the enhancement New feature or request label May 20, 2025
@nscuro nscuro merged commit 71daa02 into DependencyTrack:main May 20, 2025
4 checks passed
steinwelberg pushed a commit to steinwelberg/dependency-track-client-go that referenced this pull request May 20, 2025
@SolarFactories @steinwelberg
feat: Add TagService (DependencyTrack#45)
d2bd09f
@SolarFactories SolarFactories deleted the tag branch May 22, 2025 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nscuro nscuro nscuro approved these changes

Assignees

No one assigned

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@SolarFactories @owasp-dt-bot @nscuro