dependency-track icon indicating copy to clipboard operation
dependency-track copied to clipboard

Added projectUuid via MDC to logger statements within VEX upload.

Open ElenaStroebele opened this issue 2 weeks ago • 3 comments

Description

Added MDC based logging for VEX upload to include projectUuid in logger statements and fixed a tiny little typo. Added basic test for the VexUploadProcessingTask.java.

Addressed Issue

Addressed issue #5608 .

Additional Details

Used ChatGPT and Copilot to understand existing codebase and suggest changes.

Checklist

  • [x] 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
  • [x] 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

ElenaStroebele avatar Dec 09 '25 08:12 ElenaStroebele

:white_check_mark: Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
:white_check_mark: Open Source Security 0 0 0 0 0 issues

:computer: Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

owasp-dt-bot avatar Dec 09 '25 08:12 owasp-dt-bot

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
:white_check_mark: +0.29% (target: -1.00%) :white_check_mark: 100.00% (target: 70.00%)
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (f325bebaed5ecd8bf8f9c821b3f68575fdade86e) 24092 19492 80.91%
Head commit (acdd764925832e14c2e7d7df3d14d747422b676e) 24094 (+2) 19563 (+71) 81.19% (+0.29%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#5615) 5 5 100.00%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

codacy-production[bot] avatar Dec 09 '25 09:12 codacy-production[bot]

@nscuro Please consider this PR in release 4.13.7.

ElenaStroebele avatar Dec 10 '25 15:12 ElenaStroebele