Added projectUuid via MDC to logger statements within VEX upload.
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
:white_check_mark: Snyk checks have passed. No issues have been found so far.
| Status | Scanner | 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.
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
@nscuro Please consider this PR in release 4.13.7.