Skip to content
This repository was archived by the owner on Jun 29, 2026. It is now read-only.

fix: use correct project parameter for VEX upload endpoint - #23

Merged
nvima merged 1 commit into
mainfrom
fix/vex-upload-parameter
Jul 21, 2025
Merged

fix: use correct project parameter for VEX upload endpoint#23
nvima merged 1 commit into
mainfrom
fix/vex-upload-parameter

Conversation

@nvima

@nvima nvima commented Jul 18, 2025

Copy link
Copy Markdown
Contributor

The VEX upload endpoint was using an incorrect parameter name projectUUID instead of the expected project parameter.

https://yoursky.blue/documentation/rest-api#tag/vex/operation/uploadVex

@nvima
nvima requested a review from Copilot July 18, 2025 13:47
@nvima nvima self-assigned this Jul 18, 2025
@nvima nvima added the bug Something isn't working label Jul 18, 2025

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes an incorrect parameter name in the VEX upload endpoint to align with the API specification. The parameter was incorrectly named projectUUID when it should be project.

  • Updates parameter name from projectUUID to project in the VEX upload form data

Comment thread src/main/kotlin/com/liftric/dtcp/service/DependencyTrack.kt
@nvima
nvima requested review from Ingwersaft and tokawpl July 18, 2025 13:49

@tokawpl tokawpl left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@nvima
nvima requested review from Khartris and removed request for Ingwersaft July 18, 2025 13:53

@Khartris Khartris left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@nvima
nvima merged commit c28e82a into main Jul 21, 2025
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants