Skip to content
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

[Backport][ipa-4-11] Vault: add additional fallback to RSA-OAEP wrapping algo #7261

Closed
wants to merge 1 commit into from

Conversation

rcritten
Copy link
Contributor

@rcritten rcritten commented Mar 5, 2024

This PR was opened automatically because PR #7259 was pushed to master and backport to ipa-4-11 is required.

There is a fallback when creating the wrapping key but one was missing
when trying to use the cached transport_cert.

This allows, along with forcing keyWrap.useOAEP=true, vault creation
on an nCipher HSM.

This can be seen in HSMs where the device doesn't support the
PKCS#1 v1.5 mechanism. It will error out with either "invalid
algorithm" or CKR_FUNCTION_FAILED.

Related: https://pagure.io/freeipa/issue/9191

Signed-off-by: Rob Crittenden <rcritten@redhat.com>
@rcritten rcritten added the ack Pull Request approved, can be merged label Mar 5, 2024
@rcritten
Copy link
Contributor Author

rcritten commented Mar 5, 2024

PR was ACKed automatically because this is backport of PR #7259. Wait for CI to finish before pushing. In case of questions or problems contact @rcritten who is author of the original PR.

@flo-renaud flo-renaud added the pushed Pull Request has already been pushed label Mar 6, 2024
@flo-renaud
Copy link
Contributor

ipa-4-11:

  • ca561f7 Vault: add additional fallback to RSA-OAEP wrapping algo

@flo-renaud flo-renaud closed this Mar 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
2 participants