-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Add support for AES encryption and key wrap algorithms #16025
Conversation
Resolves Azure#14887
sdk/keyvault/Azure.Security.KeyVault.Keys/src/Cryptography/EncryptOptions.cs
Outdated
Show resolved
Hide resolved
Note this is only for feedback. Once we have some initial consensus I'll add unit tests, work on adding recorded tests (we have an an environment I should be able to access), and leverage what algorithms we may from track 1 if they have been reviewed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - Added some nits and minor suggestions.
sdk/keyvault/Azure.Security.KeyVault.Keys/src/Cryptography/CryptographyClient.cs
Show resolved
Hide resolved
sdk/keyvault/Azure.Security.KeyVault.Keys/src/Cryptography/CryptographyClient.cs
Show resolved
Hide resolved
sdk/keyvault/Azure.Security.KeyVault.Keys/src/Cryptography/CryptographyClient.cs
Outdated
Show resolved
Hide resolved
sdk/keyvault/Azure.Security.KeyVault.Keys/src/Cryptography/LocalCryptographyClient.cs
Show resolved
Hide resolved
sdk/keyvault/Azure.Security.KeyVault.Keys/api/Azure.Security.KeyVault.Keys.netstandard2.0.cs
Outdated
Show resolved
Hide resolved
* Add oct-HSM key type Resolves Azure#14887 * Add additional encryption algorithms to Keys Resolves Azure#14888 * Add AES-CBC and AES-GCM implementations/proxies * Add AES support to AesCryptographyProvider * Use factory methods for encrypt/decrypt options * Update public API
Resolves #14887 and resolves #14888