Skip to content

chore: remove stale security-override package pins - #438

Merged
BenjaminMichaelis merged 1 commit into
mainfrom
copilot/remove-explicit-package-references
Aug 16, 2026
Merged

chore: remove stale security-override package pins#438
BenjaminMichaelis merged 1 commit into
mainfrom
copilot/remove-explicit-package-references

Conversation

Copilot AI commented Aug 16, 2026

Copy link
Copy Markdown

System.Net.Http (4.3.4) and System.Security.Cryptography.X509Certificates (4.3.2) were pinned in Directory.Packages.props to override vulnerable transitive versions, but neither is directly referenced in any .csproj. On .NET 10 these pins are dead weight — the runtime ships safe versions and the old 4.x packages are no longer pulled transitively.

Removed

  • System.Net.Http 4.3.4
  • System.Security.Cryptography.X509Certificates 4.3.2

…p and System.Security.Cryptography.X509Certificates

Co-authored-by: BenjaminMichaelis <22186029+BenjaminMichaelis@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes obsolete central package version pins that were previously used as security overrides for transitive dependencies, aligning the repo with .NET 10 where these 4.x packages are no longer pulled transitively.

Changes:

  • Removed the System.Net.Http (4.3.4) central package pin.
  • Removed the System.Security.Cryptography.X509Certificates (4.3.2) central package pin.

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

@BenjaminMichaelis
BenjaminMichaelis merged commit bc934a5 into main Aug 16, 2026
11 checks passed
@BenjaminMichaelis
BenjaminMichaelis deleted the copilot/remove-explicit-package-references branch August 16, 2026 13:13
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.

3 participants