Skip to content

Conversation

@rzikm
Copy link
Member

@rzikm rzikm commented Sep 11, 2025

Closes #119483

Copilot AI review requested due to automatic review settings September 11, 2025 16:33
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 disables RSA padding tests on OpenSSL versions prior to 3.0 to resolve test compatibility issues. The change adds OpenSSL version checks to skip tests that require OpenSSL 3.0+ functionality.

  • Adds OpenSSL version validation to two RSA padding test methods
  • Throws SkipTestException when OpenSSL version is below 3.0

@rzikm rzikm changed the title Disable RSA padding tests on OpenSSL versions prior to 3.0 Disable DisableUnusedRsaPadding_Connects/DisableUsedRsaPadding_Throws on OpenSSL 1.1.1 Sep 11, 2025
@dotnet-policy-service
Copy link
Contributor

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

Copy link
Member

@wfurt wfurt left a comment

Choose a reason for hiding this comment

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

LGTM

@rzikm rzikm merged commit 094da3a into dotnet:main Sep 12, 2025
88 of 90 checks passed
@tmds
Copy link
Member

tmds commented Sep 15, 2025

@rzikm thank you! I assume this isn't something you'd consider backporting to 10.0 branch?

@rzikm
Copy link
Member Author

rzikm commented Sep 15, 2025

/backport to release/10.0

@github-actions
Copy link
Contributor

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

DisableUnusedRsaPadding_Connects/DisableUsedRsaPadding_Throws tests fail on RHEL 8

3 participants