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

Switch to checking for AES_KEY_WRAP by default #677

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cipherboy
Copy link
Member

Signed-off-by: Alexander Scheel <[email protected]>

Code I had lying around, should probably be merged at some point in the next release cycle. Was pending discussions with Jack.

@fmarco76
Copy link
Member

@cipherboy this PR seems straight-forward. Do you remember the pending discussion and/or the reason it is not merged?

@cipherboy
Copy link
Member Author

cipherboy commented May 29, 2023 via email

@fmarco76
Copy link
Member

I think on my NSS fork, there's an issue discussing this.

I found the PR in your NSS fork about key mobility to/from HSM. The current NSS code has evolved and there are differences so I need to investigate more on this. Thanks!

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

Successfully merging this pull request may close these issues.

2 participants