Handle OSSL 3.4 change to SAN:othername formatting by bartonjs · Pull Request #111820 · dotnet/runtime

@bartonjs

@bartonjs

Choose a reason for hiding this comment

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

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

Comments suppressed due to low confidence (1)

src/libraries/System.Security.Cryptography/tests/AsnEncodedDataTests.cs:121

  • The error message string contains double colons '::' which seems to be a typo. It should be 'othername: UPN:subjectupn1@example.org'.
isOpenSsl3 ? "othername: UPN::subjectupn1@example.org" : "othername:<unsupported>",

vcsjones

This was referenced

Jan 25, 2025

@vcsjones

For what it's worth, the commit / PR title mention SAN, but this is for UPN.

Okay never mind. It's the UPN in the SAN.

@vcsjones

/backport to release/9.0-staging

@vcsjones

/backport to release/8.0-staging

@github-actions

@github-actions

@github-actions github-actions bot locked as resolved and limited conversation to collaborators

May 7, 2025

@vcsjones

8.0 backport failed, but it didn't make a comment because the 9.0 re-locked the PR before 8.0 finished running.

I'll do 8.0 manually in a bit.