Skip to content

Conversation

@ericstj
Copy link
Member

@ericstj ericstj commented Oct 30, 2025

Official build test -- https://dev.azure.com/dnceng/internal/_build/results?buildId=2828070&view=results

Note the build will fail due to #6988 but should run enough for us to see binskim results.

Microsoft Reviewers: Open in CodeFlow

Copy link
Contributor

Copilot AI left a comment

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 enhances security scanning capabilities in the Azure DevOps pipeline by configuring Guardian (GDN) extraction tools and enabling comprehensive BinSkim scanning.

Key changes:

  • Adds GDN extraction configuration for security analysis tools (BinSkim, Bandit, Roslyn Analyzers)
  • Configures file type filters for extracting artifacts from build outputs
  • Enables BinSkim scanning for all file extensions through feature flags

@ericstj ericstj marked this pull request as draft October 31, 2025 01:07
Previously I set the pipeline to publish all artifacts, which was way too much, since that contained referenced binaries, tests, etc.  Instead we need to publish only what we build and ship from this repo.  That way we only run validation on those shipping packages.
@ericstj ericstj marked this pull request as ready for review October 31, 2025 15:47
@joperezr
Copy link
Member

Thanks a lot @ericstj!

@ericstj
Copy link
Member Author

ericstj commented Oct 31, 2025

Not quite ready, I think I mistook the path

Path does not exist: D:\a_work\1\a\artifacts\packages

@ericstj
Copy link
Member Author

ericstj commented Nov 4, 2025

Should be better now - https://dev.azure.com/dnceng/internal/_build/results?buildId=2830669&view=results

@ericstj ericstj merged commit a2d763f into dotnet:main Nov 4, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants