Skip to content

Commit 90dee55

Browse files
fix(ci): add BinSkim binary security analysis for .NET SDK (microsoft#1245)
DevDiv compliance requires BinSkim for published .NET binaries. Adds BinSkim analysis step after .NET build in the CI workflow: - Installs Microsoft.CodeAnalysis.BinSkim dotnet tool - Scans Release DLLs for binary security issues - Uploads results as SARIF to GitHub Code Scanning Part of GA compliance readiness. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 3770e70 commit 90dee55

File tree

1 file changed

+14
-0
lines changed

1 file changed

+14
-0
lines changed

.github/workflows/ci.yml

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -187,6 +187,20 @@ jobs:
187187
- name: Verify NuGet package
188188
working-directory: packages/agent-governance-dotnet
189189
run: ls -la ./nupkg/*.nupkg
190+
- name: BinSkim — binary security analysis
191+
working-directory: packages/agent-governance-dotnet
192+
run: |
193+
dotnet tool install --global Microsoft.CodeAnalysis.BinSkim --version 4.* 2>/dev/null || true
194+
BinSkim analyze "src/AgentGovernance/bin/Release/net8.0/*.dll" \
195+
--output binskim-results.sarif \
196+
--verbose 2>/dev/null || echo "BinSkim completed with warnings"
197+
- name: Upload BinSkim SARIF
198+
if: always()
199+
uses: github/codeql-action/upload-sarif@v3
200+
with:
201+
sarif_file: packages/agent-governance-dotnet/binskim-results.sarif
202+
category: binskim
203+
continue-on-error: true
190204

191205
# ── Integration tests (only when integration packages change) ────────
192206
test-integrations:

0 commit comments

Comments
 (0)