feat: Add scan IDs to text and SARIF output formats - #665
Jiri-Stary wants to merge 10 commits into
Conversation
- Add ScanIds field to ScaScanResults structure to store scan IDs from scan responses
- Collect scan IDs when processing SCA scan results
- Add ScanIdSarifPropertyKey constant to SARIF utils
- Store scan IDs in SARIF run properties for SCA scans (comma-separated)
- Add hostedViewerUri to SARIF results linking to on-demand scanning page
- Display scan IDs with on-demand scanning links in text output
- Links are generated in format: {platformUrl}ui/onDemandScanning/{scanId}
- Support both single and multiple scan ID display
- JSON output includes scan IDs through ScanResponse.ScanId and ScaScanResults.ScanIds fields
Fixes jfrog#199
|
All contributors have signed the CLA ✍️ ✅ |
|
I have read the CLA Document and I hereby sign the CLA |
|
@eranturgeman Hello, this is my first contribution. Have i missed anything in the contributuing guidelines ? I have added this here in Feb and had no response since. Can you give me any advice please? |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
Fixes #199
devbranch.go vet ./....go fmt ./....