Skip to content

Fix Dependency Track Hook #3290

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 1 commit into from
Sep 30, 2025
Merged

Fix Dependency Track Hook #3290

merged 1 commit into from
Sep 30, 2025

Conversation

@p4trickweiss
Copy link
Contributor

@p4trickweiss p4trickweiss commented Sep 24, 2025 •
edited
Loading

Description

Since getRawResults() from the hook-sdk returns a string, it is necessary to parse the value to an object in the hook.

(closes #3272)

Checklist

  • Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
  • Make sure that all your commits are signed-off and that you are added to the Contributors file.
  • Make sure that all CI finish successfully.
  • Optional (but appreciated): Make sure that all commits are Verified.

@p4trickweiss
parse response string to object
e68ec18 
Signed-off-by: Patrick <patrick.weiss@iteratec.com>
@netlify
Copy link

netlify bot commented Sep 24, 2025 •
edited
Loading

✅ Deploy Preview for docs-securecodebox canceled.

Name Link
🔨 Latest commit e68ec18
🔍 Latest deploy log https://app.netlify.com/projects/docs-securecodebox/deploys/68d3c9a240df4c0009c47b90

@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 24, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@p4trickweiss p4trickweiss requested a review from J12934 September 24, 2025 10:38
@p4trickweiss p4trickweiss moved this from Triage to To Review in secureCodeBox Sep 24, 2025
@p4trickweiss p4trickweiss marked this pull request as ready for review September 30, 2025 09:00
@github-project-automation github-project-automation bot moved this from To Review to Reviewer Approved in secureCodeBox Sep 30, 2025
@J12934 J12934 merged commit c73801b into secureCodeBox:main Sep 30, 2025
49 checks passed
@github-project-automation github-project-automation bot moved this from Reviewer Approved to Done in secureCodeBox Sep 30, 2025
@p4trickweiss p4trickweiss deleted the fix/dependencytrack branch October 1, 2025 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@J12934 J12934 J12934 approved these changes

Assignees

@p4trickweiss p4trickweiss

Labels

None yet

Projects

secureCodeBox
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Dependency-Track hook ignores CycloneDX SBOM after secureCodeBox v5.0.0 upgrade

2 participants

@p4trickweiss @J12934