Skip to content

[release/9.0-staging] fix: in rsa signatures, configure digest before padding mode #115695

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: release/9.0-staging
Choose a base branch
from

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented May 18, 2025

Backport of #114261 to release/9.0-staging

/cc @vcsjones @rcatolino

Customer Impact

  • Customer reported
  • Found internally

This was originally reported by a customer in #114260, and a request to back ported it was made in #115693.

Customers using RSA.SignHash with RSASSA-PSS on Linux which is locked-down to FIPS are unable to use it because the configuration was done in such a way that confused the FIPS-only validation logic in OpenSSL. Customers have no work-arounds for this, other than do disable the FIPS enforcement on Linux.

Regression

  • Yes
  • No

No. It's been like this since we moved off the managed implementation of PSS padding for Linux.

Testing

This was manually validated on an Ubuntu Pro installation that was FIPS enforced.

Risk

Low. The code that configures the signing context has only changed in the order in which it does the configuration. The area has strong unit test covered to ensure existing scenarios did not regress.

IMPORTANT: If this backport is for a servicing release, please verify that:

  • The PR target branch is release/X.0-staging, not release/X.0.

Package authoring no longer needed in .NET 9

IMPORTANT: Starting with .NET 9, you no longer need to edit a NuGet package's csproj to enable building and bump the version.
Keep in mind that we still need package authoring in .NET 8 and older versions.

@vcsjones vcsjones requested a review from bartonjs May 18, 2025 17:25
@vcsjones vcsjones self-assigned this May 18, 2025
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.

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

Successfully merging this pull request may close these issues.

2 participants