-
Notifications
You must be signed in to change notification settings - Fork 3
feat(sdk): Move to rewrap v2 request/response format #774
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
Conversation
|
If these changes look good, signoff on them with: If they aren't any good, please remove them with: |
X-Test Failure Resultsopentdf-sdk-lib |
X-Test Failure Resultsopentdf-sdk-lib |
* - Pin `@bufbuild/buf` and `@bufbuild/protoc-gen-es` dependencies to specific versions. - Update copyright notices in `http_pb.ts` and `validate_pb.ts`. * - Pin `@bufbuild/buf` and `@bufbuild/protoc-gen-es` dependencies to specific versions. - Update copyright notices in `http_pb.ts` and `validate_pb.ts`. * Add `obligations` and `rootCerts` attributes to test fixtures and mock data. Signed-off-by: Elizabeth Healy <ehealy@virtru.com>
Signed-off-by: Elizabeth Healy <ehealy@virtru.com>
Signed-off-by: Elizabeth Healy <ehealy@virtru.com>
* - Pin `@bufbuild/buf` and `@bufbuild/protoc-gen-es` dependencies to specific versions. - Update copyright notices in `http_pb.ts` and `validate_pb.ts`. * - Pin `@bufbuild/buf` and `@bufbuild/protoc-gen-es` dependencies to specific versions. - Update copyright notices in `http_pb.ts` and `validate_pb.ts`. * Add `obligations` and `rootCerts` attributes to test fixtures and mock data. * Add `getRootCertsFromNamespace` function and include headers initialization in `authProvider` * Add input validation for `getRootCertsFromNamespace` and basic unit tests Signed-off-by: Elizabeth Healy <ehealy@virtru.com>
* feat(core): initial obligations support in rewrap flow * 🤖 🎨 Autoformat Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * wip Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * more wip * rm unused import * 🤖 🎨 Autoformat Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * lint fix Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * tests Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * 🤖 🎨 Autoformat Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * move file * tdf3 client * cleanup * obligations method on opentdf reader classes * requiredObligations on DecoratedReadableStream in tdf3 * wip: fetch decision if obligations haven't been set on reader * wip * 🤖 🎨 Autoformat Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * bugfix in case of no data attributes leading to no obligations Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * working state Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * fix Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * 🤖 🎨 Autoformat Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * fix comments * rm example web app hardcoded attributes and obligations * unit tests for getRequiredObligations * improve nullish operators * cleanup * 🤖 🎨 Autoformat Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * improvements * fix * 🤖 🎨 Autoformat Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * improve log * put back package.json changes * pr feedback Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * 🤖 🎨 Autoformat Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * rm rewrap header for obligations over legacy http for older platforms --------- Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> Signed-off-by: Elizabeth Healy <ehealy@virtru.com>
* chore(main): release sdk 0.5.0 * Update dependencies --------- Co-authored-by: opentdf-automation[bot] <149537512+opentdf-automation[bot]@users.noreply.github.com> Signed-off-by: Elizabeth Healy <ehealy@virtru.com>
…l/go versions (#767) * add a workflow to update the pbs * trigger on PR * correct platform location * add gh token to env * remove extra file after use * detect changes on regen * test with latest version * remove, test changes * test for signed commits * try with api * push the new branch * use a shorter file name in the message * fix for non existing files * run slightly after midnight to avoid queues Signed-off-by: Elizabeth Healy <ehealy@virtru.com>
Signed-off-by: Elizabeth Healy <ehealy@virtru.com>
Signed-off-by: Elizabeth Healy <ehealy@virtru.com>
Signed-off-by: Elizabeth Healy <ehealy@virtru.com>
Signed-off-by: Elizabeth Healy <ehealy@virtru.com>
…763) Bumps [playwright](https://github.com/microsoft/playwright) to 1.56.1 and updates ancestor dependency [@playwright/test](https://github.com/microsoft/playwright). These dependencies need to be updated together. Updates `playwright` from 1.50.1 to 1.56.1 - [Release notes](https://github.com/microsoft/playwright/releases) - [Commits](microsoft/playwright@v1.50.1...v1.56.1) Updates `@playwright/test` from 1.50.1 to 1.56.1 - [Release notes](https://github.com/microsoft/playwright/releases) - [Commits](microsoft/playwright@v1.50.1...v1.56.1) --- updated-dependencies: - dependency-name: playwright dependency-version: 1.56.1 dependency-type: indirect - dependency-name: "@playwright/test" dependency-version: 1.56.1 dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Signed-off-by: Elizabeth Healy <ehealy@virtru.com>
Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 6.3.6 to 6.4.1. - [Release notes](https://github.com/vitejs/vite/releases) - [Changelog](https://github.com/vitejs/vite/blob/main/packages/vite/CHANGELOG.md) - [Commits](https://github.com/vitejs/vite/commits/create-vite@6.4.1/packages/vite) --- updated-dependencies: - dependency-name: vite dependency-version: 6.4.1 dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Signed-off-by: Elizabeth Healy <ehealy@virtru.com>
Signed-off-by: Elizabeth Healy <ehealy@virtru.com>
Signed-off-by: Elizabeth Healy <ehealy@virtru.com>
Bumps [playwright](https://github.com/microsoft/playwright) and [@playwright/test](https://github.com/microsoft/playwright). These dependencies needed to be updated together. Updates `playwright` from 1.50.1 to 1.56.1 - [Release notes](https://github.com/microsoft/playwright/releases) - [Commits](microsoft/playwright@v1.50.1...v1.56.1) Updates `@playwright/test` from 1.50.1 to 1.56.1 - [Release notes](https://github.com/microsoft/playwright/releases) - [Commits](microsoft/playwright@v1.50.1...v1.56.1) --- updated-dependencies: - dependency-name: playwright dependency-version: 1.56.1 dependency-type: direct:development - dependency-name: "@playwright/test" dependency-version: 1.56.1 dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Signed-off-by: Elizabeth Healy <ehealy@virtru.com>
ae82fc7 to
347302e
Compare
X-Test Failure Reportopentdf-sdk-lib |
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.
Pull Request Overview
This PR migrates the rewrap request/response format from v1 (deprecated) to v2 (bulk-style). The changes update the SDK to use protobuf-based request/response structures while maintaining backward compatibility through an upgrade function. The implementation doesn't introduce actual bulk support but adopts the bulk-style format to align with ACP requirements.
Key Changes
- Replaced JSON-based rewrap requests with protobuf
UnsignedRewrapRequeststructures - Updated rewrap response handling to process
RewrapResponsewith nested result structures - Added
upgradeRewrapResponseV1function to handle backward compatibility with v1 responses
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| lib/tests/server.ts | Updated mock KAS server to parse/generate v2 protobuf rewrap requests/responses |
| lib/tdf3/src/tdf.ts | Converted TDF3 rewrap logic to use protobuf request format and handle v2 response structure |
| lib/src/utils.ts | Added upgradeRewrapResponseV1 helper function for v1-to-v2 response migration |
| lib/src/nanotdf/Client.ts | Updated NanoTDF client to build protobuf rewrap requests and process v2 responses |
| lib/src/access/access-rpc.ts | Enhanced error handling with new handleRpcRewrapError and handleRpcRewrapErrorString functions |
| .github/workflows/format.yaml | Commented out workflow trigger (likely temporary for development) |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
eugenioenko
left a comment
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.
PR changes look great! 🎉
Left a few non-blocking questions and comments that can be addressed later on
* feat: Certificates & Obligations (#755) * - Pin `@bufbuild/buf` and `@bufbuild/protoc-gen-es` dependencies to specific versions. - Update copyright notices in `http_pb.ts` and `validate_pb.ts`. * - Pin `@bufbuild/buf` and `@bufbuild/protoc-gen-es` dependencies to specific versions. - Update copyright notices in `http_pb.ts` and `validate_pb.ts`. * Add `obligations` and `rootCerts` attributes to test fixtures and mock data. Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * feat: upgrade tdf clients to rewrap v2 proto structure Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * updates to match go behavior Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * feat: Get Namespace (#756) * - Pin `@bufbuild/buf` and `@bufbuild/protoc-gen-es` dependencies to specific versions. - Update copyright notices in `http_pb.ts` and `validate_pb.ts`. * - Pin `@bufbuild/buf` and `@bufbuild/protoc-gen-es` dependencies to specific versions. - Update copyright notices in `http_pb.ts` and `validate_pb.ts`. * Add `obligations` and `rootCerts` attributes to test fixtures and mock data. * Add `getRootCertsFromNamespace` function and include headers initialization in `authProvider` * Add input validation for `getRootCertsFromNamespace` and basic unit tests Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * feat(sdk): initial obligations support in rewrap flow (#748) * feat(core): initial obligations support in rewrap flow * 🤖 🎨 Autoformat Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * wip Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * more wip * rm unused import * 🤖 🎨 Autoformat Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * lint fix Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * tests Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * 🤖 🎨 Autoformat Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * move file * tdf3 client * cleanup * obligations method on opentdf reader classes * requiredObligations on DecoratedReadableStream in tdf3 * wip: fetch decision if obligations haven't been set on reader * wip * 🤖 🎨 Autoformat Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * bugfix in case of no data attributes leading to no obligations Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * working state Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * fix Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * 🤖 🎨 Autoformat Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * fix comments * rm example web app hardcoded attributes and obligations * unit tests for getRequiredObligations * improve nullish operators * cleanup * 🤖 🎨 Autoformat Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * improvements * fix * 🤖 🎨 Autoformat Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * improve log * put back package.json changes * pr feedback Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * 🤖 🎨 Autoformat Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * rm rewrap header for obligations over legacy http for older platforms --------- Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * chore: release sdk 0.5.0 (#658) * chore(main): release sdk 0.5.0 * Update dependencies --------- Co-authored-by: opentdf-automation[bot] <149537512+opentdf-automation[bot]@users.noreply.github.com> Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * feat(ci): Add a workflow to update the generated code for new protocol/go versions (#767) * add a workflow to update the pbs * trigger on PR * correct platform location * add gh token to env * remove extra file after use * detect changes on regen * test with latest version * remove, test changes * test for signed commits * try with api * push the new branch * use a shorter file name in the message * fix for non existing files * run slightly after midnight to avoid queues Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * 🤖 🎨 Autoformat Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * handle rewrap response Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * formatting Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * passing unit tests Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * format Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * chore(docs): bump playwright and @playwright/test in /web-app/tests (#763) Bumps [playwright](https://github.com/microsoft/playwright) to 1.56.1 and updates ancestor dependency [@playwright/test](https://github.com/microsoft/playwright). These dependencies need to be updated together. Updates `playwright` from 1.50.1 to 1.56.1 - [Release notes](https://github.com/microsoft/playwright/releases) - [Commits](microsoft/playwright@v1.50.1...v1.56.1) Updates `@playwright/test` from 1.50.1 to 1.56.1 - [Release notes](https://github.com/microsoft/playwright/releases) - [Commits](microsoft/playwright@v1.50.1...v1.56.1) --- updated-dependencies: - dependency-name: playwright dependency-version: 1.56.1 dependency-type: indirect - dependency-name: "@playwright/test" dependency-version: 1.56.1 dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * chore(docs): bump vite from 6.3.6 to 6.4.1 in /web-app (#764) Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 6.3.6 to 6.4.1. - [Release notes](https://github.com/vitejs/vite/releases) - [Changelog](https://github.com/vitejs/vite/blob/main/packages/vite/CHANGELOG.md) - [Commits](https://github.com/vitejs/vite/commits/create-vite@6.4.1/packages/vite) --- updated-dependencies: - dependency-name: vite dependency-version: 6.4.1 dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * v1 backwards compatability Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * error handling Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * cleanup Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * chore(docs): bump playwright and @playwright/test in /web-app (#775) Bumps [playwright](https://github.com/microsoft/playwright) and [@playwright/test](https://github.com/microsoft/playwright). These dependencies needed to be updated together. Updates `playwright` from 1.50.1 to 1.56.1 - [Release notes](https://github.com/microsoft/playwright/releases) - [Commits](microsoft/playwright@v1.50.1...v1.56.1) Updates `@playwright/test` from 1.50.1 to 1.56.1 - [Release notes](https://github.com/microsoft/playwright/releases) - [Commits](microsoft/playwright@v1.50.1...v1.56.1) --- updated-dependencies: - dependency-name: playwright dependency-version: 1.56.1 dependency-type: direct:development - dependency-name: "@playwright/test" dependency-version: 1.56.1 dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * suggestions --------- Signed-off-by: Elizabeth Healy <ehealy@virtru.com> Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Paul Flynn <43211074+pflynn-virtru@users.noreply.github.com> Co-authored-by: jakedoublev <jake.vanvorhis@virtru.com> Co-authored-by: Jake Van Vorhis <83739412+jakedoublev@users.noreply.github.com> Co-authored-by: opentdf-automation[bot] <149537512+opentdf-automation[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Signed-off-by: Elizabeth Healy <ehealy@virtru.com>
* feat(sdk): Move to rewrap v2 request/response format (#774) * feat: Certificates & Obligations (#755) * - Pin `@bufbuild/buf` and `@bufbuild/protoc-gen-es` dependencies to specific versions. - Update copyright notices in `http_pb.ts` and `validate_pb.ts`. * - Pin `@bufbuild/buf` and `@bufbuild/protoc-gen-es` dependencies to specific versions. - Update copyright notices in `http_pb.ts` and `validate_pb.ts`. * Add `obligations` and `rootCerts` attributes to test fixtures and mock data. Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * feat: upgrade tdf clients to rewrap v2 proto structure Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * updates to match go behavior Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * feat: Get Namespace (#756) * - Pin `@bufbuild/buf` and `@bufbuild/protoc-gen-es` dependencies to specific versions. - Update copyright notices in `http_pb.ts` and `validate_pb.ts`. * - Pin `@bufbuild/buf` and `@bufbuild/protoc-gen-es` dependencies to specific versions. - Update copyright notices in `http_pb.ts` and `validate_pb.ts`. * Add `obligations` and `rootCerts` attributes to test fixtures and mock data. * Add `getRootCertsFromNamespace` function and include headers initialization in `authProvider` * Add input validation for `getRootCertsFromNamespace` and basic unit tests Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * feat(sdk): initial obligations support in rewrap flow (#748) * feat(core): initial obligations support in rewrap flow * 🤖 🎨 Autoformat Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * wip Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * more wip * rm unused import * 🤖 🎨 Autoformat Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * lint fix Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * tests Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * 🤖 🎨 Autoformat Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * move file * tdf3 client * cleanup * obligations method on opentdf reader classes * requiredObligations on DecoratedReadableStream in tdf3 * wip: fetch decision if obligations haven't been set on reader * wip * 🤖 🎨 Autoformat Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * bugfix in case of no data attributes leading to no obligations Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * working state Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * fix Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * 🤖 🎨 Autoformat Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * fix comments * rm example web app hardcoded attributes and obligations * unit tests for getRequiredObligations * improve nullish operators * cleanup * 🤖 🎨 Autoformat Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * improvements * fix * 🤖 🎨 Autoformat Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * improve log * put back package.json changes * pr feedback Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * 🤖 🎨 Autoformat Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> * rm rewrap header for obligations over legacy http for older platforms --------- Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * chore: release sdk 0.5.0 (#658) * chore(main): release sdk 0.5.0 * Update dependencies --------- Co-authored-by: opentdf-automation[bot] <149537512+opentdf-automation[bot]@users.noreply.github.com> Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * feat(ci): Add a workflow to update the generated code for new protocol/go versions (#767) * add a workflow to update the pbs * trigger on PR * correct platform location * add gh token to env * remove extra file after use * detect changes on regen * test with latest version * remove, test changes * test for signed commits * try with api * push the new branch * use a shorter file name in the message * fix for non existing files * run slightly after midnight to avoid queues Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * 🤖 🎨 Autoformat Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * handle rewrap response Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * formatting Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * passing unit tests Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * format Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * chore(docs): bump playwright and @playwright/test in /web-app/tests (#763) Bumps [playwright](https://github.com/microsoft/playwright) to 1.56.1 and updates ancestor dependency [@playwright/test](https://github.com/microsoft/playwright). These dependencies need to be updated together. Updates `playwright` from 1.50.1 to 1.56.1 - [Release notes](https://github.com/microsoft/playwright/releases) - [Commits](microsoft/playwright@v1.50.1...v1.56.1) Updates `@playwright/test` from 1.50.1 to 1.56.1 - [Release notes](https://github.com/microsoft/playwright/releases) - [Commits](microsoft/playwright@v1.50.1...v1.56.1) --- updated-dependencies: - dependency-name: playwright dependency-version: 1.56.1 dependency-type: indirect - dependency-name: "@playwright/test" dependency-version: 1.56.1 dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * chore(docs): bump vite from 6.3.6 to 6.4.1 in /web-app (#764) Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 6.3.6 to 6.4.1. - [Release notes](https://github.com/vitejs/vite/releases) - [Changelog](https://github.com/vitejs/vite/blob/main/packages/vite/CHANGELOG.md) - [Commits](https://github.com/vitejs/vite/commits/create-vite@6.4.1/packages/vite) --- updated-dependencies: - dependency-name: vite dependency-version: 6.4.1 dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * v1 backwards compatability Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * error handling Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * cleanup Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * chore(docs): bump playwright and @playwright/test in /web-app (#775) Bumps [playwright](https://github.com/microsoft/playwright) and [@playwright/test](https://github.com/microsoft/playwright). These dependencies needed to be updated together. Updates `playwright` from 1.50.1 to 1.56.1 - [Release notes](https://github.com/microsoft/playwright/releases) - [Commits](microsoft/playwright@v1.50.1...v1.56.1) Updates `@playwright/test` from 1.50.1 to 1.56.1 - [Release notes](https://github.com/microsoft/playwright/releases) - [Commits](microsoft/playwright@v1.50.1...v1.56.1) --- updated-dependencies: - dependency-name: playwright dependency-version: 1.56.1 dependency-type: direct:development - dependency-name: "@playwright/test" dependency-version: 1.56.1 dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * suggestions --------- Signed-off-by: Elizabeth Healy <ehealy@virtru.com> Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Paul Flynn <43211074+pflynn-virtru@users.noreply.github.com> Co-authored-by: jakedoublev <jake.vanvorhis@virtru.com> Co-authored-by: Jake Van Vorhis <83739412+jakedoublev@users.noreply.github.com> Co-authored-by: opentdf-automation[bot] <149537512+opentdf-automation[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * additional comments and cleanup Signed-off-by: Elizabeth Healy <ehealy@virtru.com> * format * fix merge --------- Signed-off-by: Elizabeth Healy <ehealy@virtru.com> Signed-off-by: jakedoublev <jake.vanvorhis@virtru.com> Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Paul Flynn <43211074+pflynn-virtru@users.noreply.github.com> Co-authored-by: jakedoublev <jake.vanvorhis@virtru.com> Co-authored-by: Jake Van Vorhis <83739412+jakedoublev@users.noreply.github.com> Co-authored-by: opentdf-automation[bot] <149537512+opentdf-automation[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>


Certain fields of the rewrap request and response have been deprecated, move to the bulk-style rewrap request/response to be in line with ACP -- the PR does not introduce bulk support, just bulk style requests
Maintains backward compatibility -- can still make and receive v1 requests and responses