Skip to content

Conversation

@elizabethhealy
Copy link
Member

@elizabethhealy elizabethhealy commented Oct 22, 2025

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

@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

@github-actions
Copy link

@github-actions
Copy link

pflynn-virtru and others added 18 commits October 24, 2025 12:53
* - 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>
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>
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>
@elizabethhealy elizabethhealy force-pushed the DSPX-1781-use-v2-rewrap branch from ae82fc7 to 347302e Compare October 24, 2025 16:56
@github-actions
Copy link

@github-actions
Copy link

@github-actions
Copy link

@elizabethhealy elizabethhealy changed the title DRAFT: use v2 rewrap feat(sdk): Move to rewrap v2 request/response format Oct 24, 2025
@github-actions
Copy link

Copy link

Copilot AI left a 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 UnsignedRewrapRequest structures
  • Updated rewrap response handling to process RewrapResponse with nested result structures
  • Added upgradeRewrapResponseV1 function 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.

@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

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

See analysis details on SonarQube Cloud

@github-actions
Copy link

@elizabethhealy elizabethhealy marked this pull request as ready for review October 24, 2025 18:44
@elizabethhealy elizabethhealy requested a review from a team as a code owner October 24, 2025 18:44
Copy link
Contributor

@eugenioenko eugenioenko left a 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

@elizabethhealy elizabethhealy merged commit e7718d5 into main Oct 28, 2025
22 of 23 checks passed
@elizabethhealy elizabethhealy deleted the DSPX-1781-use-v2-rewrap branch October 28, 2025 20:25
elizabethhealy added a commit that referenced this pull request Oct 29, 2025
* 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>
elizabethhealy added a commit that referenced this pull request Oct 29, 2025
* 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>
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.

5 participants