microsoft / sarif-azuredevops-extension-legacy

An Azure DevOps extension encapsulating the SARIF Web Component.
MIT License
8 stars 11 forks source link

Feature request: build results Scans tab should pick up Pipeline Artifacts, in addition to Build Artifacts #4

Open dbjorge opened 2 years ago

dbjorge commented 2 years ago

Issue description

Azure DevOps build pipelines support two competing types of uploaded artifacts:

In general, Azure DevOps recommends Pipeline Artifacts over Build Artifacts (see the disclaimer at the top of the Build Artifacts docs), but currently, the SARIF SAST Scans Tab extension only supports CodeAnalysisLogs artifacts created as Build Artifacts. If you attempt to upload CodeAnalysisLogs as a pipeline artifact, the Scans tab will show that no SARIF file(s) were found, and its developer console output will show a 404 error.

Ideally, the extension would check for both build and pipeline artifacts using that name and load any matching artifacts from both locations.

Motivation

This appears to be a common source of confusion:

agorischek commented 2 years ago

+1, this has been a problem for my team recently. Pipeline artifact support would be great!