Skip to content

Conversation

@dmihalcik-virtru
Copy link
Member

  • Adds parameters for requesting EC encapsulation in TDF key access objects to SDK and CLI
    • new createZTDF parameter, wrappingKeyAlgorithm. Defaults to "rsa:2048", and can now take "ec:secp256r1".
    • cli encrypt exposes this as the --encapKeyType parameter
  • Similarly, adds parameters for requesting responses with EC wrapped return values from key access server during decrypt
    • new sdk.read parameter, wrappingKeyAlgorithm. Defaults to "rsa:2048", and can now take "ec:secp256r1". Only used for ZTDF kaos currently.
    • exposes this through cli decrypt subcommand with the --rewrapKeyType parameter

@github-actions
Copy link

If these changes look good, signoff on them with:

git pull && git commit --amend --signoff && git push --force-with-lease origin

If they aren't any good, please remove them with:

git pull && git reset --hard HEAD~1 && git push --force-with-lease origin

@dmihalcik-virtru dmihalcik-virtru force-pushed the feature/ec-wrapped branch 3 times, most recently from 295e0cd to db9253a Compare February 18, 2025 21:38
@dmihalcik-virtru dmihalcik-virtru marked this pull request as ready for review February 18, 2025 22:25
@dmihalcik-virtru dmihalcik-virtru requested a review from a team as a code owner February 18, 2025 22:25
- Adds parameters for requesting EC encapsulation in TDF key access objects to SDK and CLI
  - new `createZTDF` parameter, `wrappingKeyAlgorithm`. Defaults to `"rsa:2048"`, and can now take `"ec:secp256r1"`.
  - cli `encrypt` exposes this as the `--encapKeyType` parameter
- Similarly, adds parameters for requesting responses with EC wrapped return values from key access server during decrypt
  - new `sdk.read` parameter, `wrappingKeyAlgorithm`. Defaults to `"rsa:2048"`, and can now take `"ec:secp256r1"`. Only used for ZTDF kaos currently.
  - exposes this through cli `decrypt` subcommand with the `--rewrapKeyType` parameter
This reverts commit 8708539.
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
68.4% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@dmihalcik-virtru dmihalcik-virtru merged commit 9d4eab4 into main Feb 19, 2025
12 of 13 checks passed
@dmihalcik-virtru dmihalcik-virtru deleted the feature/ec-wrapped branch February 19, 2025 16:09
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.

3 participants