Skip to content

test: pin System.Security.Cryptography.Pkcs in DiagnosticSource integration tests - #5771

Merged
jamescrosswell merged 1 commit into
mainfrom
fix/pkcs-test-pin
Oct 22, 2026
Merged

jamescrosswell merged 1 commit into
mainfrom
fix/pkcs-test-pin

Conversation

@jamescrosswell

Copy link
Copy Markdown
Collaborator

The nightly List vulnerable packages scan has failed since Oct 15 on two .NET DoS advisories against System.Security.Cryptography.Pkcs 9.0.4: GHSA-846f-f764-76mx (high) and GHSA-2fmv-rv2c-59x7 (moderate).

Only Sentry.DiagnosticSource.IntegrationTests on net10.0 is affected. It gets Pkcs transitively through Microsoft.EntityFrameworkCore.SqlServer and LocalDb, both via Microsoft.Data.SqlClient 6.1.1. None of our shipped packages reference it.

Upgrading EF Core wouldn't clear this. The latest SqlClient (6.1.7) still only requires Pkcs 9.0.11 or later on net9.0+, which is in the vulnerable range (≤ 9.0.20). So this PR pins Pkcs 10.0.13 in the net10.0 group, the same way Azure.Identity is already pinned for net9.0. This is a test project with no consumers, so pinning costs nothing, and it's better than an allowlist entry that would outlive the problem.

The SQL tests in this project only run on Windows (LocalDb), so the Windows job is the meaningful check.

#skip-changelog

🤖 Generated with Claude Code

…ration tests

Microsoft.Data.SqlClient 6.1.x pulls in Pkcs 9.0.4 on net10.0, which is
affected by GHSA-846f-f764-76mx and GHSA-2fmv-rv2c-59x7. No SqlClient
release floors Pkcs high enough to clear them yet.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@codecov

codecov Bot commented Oct 22, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.12%. Comparing base (8333e3b) to head (8791b99).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5771   +/-   ##
=======================================
  Coverage   76.12%   76.12%           
=======================================
  Files         528      528           
  Lines       19579    19579           
  Branches     3829     3829           
=======================================
+ Hits        14904    14905    +1     
  Misses       3792     3792           
+ Partials      883      882    -1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jamescrosswell
jamescrosswell marked this pull request as ready for review October 22, 2026 21:16
@github-actions github-actions Bot added the risk: low PR risk score: low label Oct 22, 2026
@jamescrosswell
jamescrosswell merged commit aa4bd9a into main Oct 22, 2026
40 checks passed
@jamescrosswell
jamescrosswell deleted the fix/pkcs-test-pin branch October 22, 2026 21:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk: low PR risk score: low

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants