-
Notifications
You must be signed in to change notification settings - Fork 55
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
Cosmos Always Encrypted misleading documentation about CMK key versions #117
base: main
Are you sure you want to change the base?
Conversation
The CreateClientEncryptionKeyAsync method requires the full path to the CMK key in KeyVault specified - it doesn't accept a key URL without the version
@asos-gaborbuzasi : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
Learn Build status updates of commit 1347f87: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
Learn Build status updates of commit a607c7e: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
Can you review the proposed changes? Important: When the changes are ready for publication, adding a #label:"aq-pr-triaged" |
This pull request has been inactive for at least 14 days. If you are finished with your changes, don't forget to sign off. See the contributor guide for instructions. |
The CreateClientEncryptionKeyAsync method requires the full path to the CMK key in KeyVault specified - it doesn't accept a key URL without the version. Thus you can't actually provide a non-version specific URL to the DEK.