Skip to content

Conversation

@vcsjones
Copy link
Member

Backport of #121450 to release/10.0

cc @bartonjs

Customer Impact

These are test only changes. These changes react to more environments in CI rejecting certificate chains that use an RSA+SHA-1 root certificate.

  • Customer reported
  • Found internally

Regression

  • Yes
  • No

Testing

Tests that were failing are now passing in CI.

Risk

None, test only changes.

@vcsjones vcsjones added this to the 10.0.x milestone Nov 15, 2025
@vcsjones vcsjones requested a review from bartonjs November 15, 2025 17:58
@vcsjones vcsjones self-assigned this Nov 15, 2025
Copilot AI review requested due to automatic review settings November 15, 2025 17:58
@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @dotnet/area-system-security, @bartonjs, @vcsjones
See info in area-owners.md if you want to be subscribed.

Copilot finished reviewing on behalf of vcsjones November 15, 2025 18:00
@vcsjones
Copy link
Member Author

For what it’s worth this backported cleanly, but the backport command could not backport from a fork because my fork had some refs that were not available in the runtime repository. The patch applied just fine when both remotes were in the index.

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 backports certificate test data updates from the main branch to release/10.0, replacing an RSA+SHA-1 certificate chain (Baltimore CyberTrust Root) with an RSA+SHA-256 certificate chain (DigiCert Global Root G2) to address CI test failures due to increasing SHA-1 rejection in test environments.

  • Updates test certificate chain from Baltimore CyberTrust Root to DigiCert Global Root G2
  • Adjusts test data including hostnames, verification times, and expected cryptographic values
  • Removes platform-specific workarounds no longer needed with the new certificate

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
MatchesHostnameTests.cs Updates test hostnames to match SANs in new certificate (e.g., www.microsoft.com.au, copilot.ai, yarp.dot.net)
AuthorityKeyIdentifierTests.cs Updates expected authority key identifier hex values to match new certificate chain
CollectionTests.cs Updates verification times from 2021-02-26 to 2025-12-25 to fall within new certificate validity period
ChainTests.cs Updates verification times throughout tests, removes SHA-1 signature support checks, simplifies platform detection logic
TestData.cs Replaces MicrosoftDotComSslCertBytes, MicrosoftDotComIssuerBytes, and MicrosoftDotComRootBytes with new certificate chain data

@bartonjs bartonjs added the Servicing-consider Issue for next servicing release review label Nov 15, 2025
@bartonjs
Copy link
Member

@artl93 Test-only change

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-System.Security Servicing-consider Issue for next servicing release review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants