Skip to content

Comments

feat: Add scan IDs to text and SARIF output formats#665

Open
Jiri-Stary wants to merge 3 commits intojfrog:devfrom
Jiri-Stary:feature/add-scan-id-to-output
Open

feat: Add scan IDs to text and SARIF output formats#665
Jiri-Stary wants to merge 3 commits intojfrog:devfrom
Jiri-Stary:feature/add-scan-id-to-output

Conversation

@Jiri-Stary
Copy link

@Jiri-Stary Jiri-Stary commented Feb 10, 2026

  • 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 #199

  • The pull request is targeting the dev branch.
  • The code has been validated to compile successfully by running go vet ./....
  • The code has been formatted properly using go fmt ./....
  • All static analysis checks passed.
  • All tests have passed. If this feature is not already covered by the tests, new tests have been added.
  • Updated the Contributing page / ReadMe page / CI Workflow files if needed.
  • All changes are detailed at the description. if not already covered at JFrog Documentation, new documentation have been added.

- 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant