Skip to content

feat: NanoTDF Implementation #46

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 29 commits into from
Jun 6, 2024
Merged

feat: NanoTDF Implementation #46

merged 29 commits into from
Jun 6, 2024

Conversation

mustyantsev
Copy link
Contributor

No description provided.

@sujankota sujankota marked this pull request as ready for review June 4, 2024 14:02
@sujankota sujankota requested review from a team as code owners June 4, 2024 14:02
ttschampel
ttschampel reviewed Jun 4, 2024
View reviewed changes
ttschampel
ttschampel reviewed Jun 4, 2024
View reviewed changes
Copy link
Member

@ttschampel ttschampel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we add a section in the README about using the Nano Encrypt feature?

patmantru
patmantru previously requested changes Jun 4, 2024
View reviewed changes
@patmantru patmantru self-requested a review June 4, 2024 16:05
patmantru
patmantru reviewed Jun 4, 2024
View reviewed changes
Copy link

@patmantru patmantru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changing to comment-only I think?

@patmantru patmantru dismissed their stale review June 4, 2024 16:08

meant this to be comment only

@ttschampel
Copy link
Member

ttschampel commented Jun 4, 2024 •
edited
Loading

Any idea why the checks aren't running for this PR? Maybe needs to be rebased off main?

@sujankota sujankota changed the title feat: SDK NanoTDF - initial commit feat: SDK NanoTDF Implementation Jun 5, 2024
@sujankota sujankota changed the title feat: SDK NanoTDF Implementation feat: NanoTDF Implementation Jun 5, 2024
patmantru
patmantru previously approved these changes Jun 5, 2024
View reviewed changes
@ttschampel ttschampel mentioned this pull request Jun 5, 2024
Merged
mkleene and others added 6 commits June 6, 2024 11:42
@mkleene @sujankota
feat(sdk): add CLI and integration tests (#64)
9abe7b1 
Add a simple CLI and set up the platform so we can run e2e tests against
it.

Also includes a fix for using encrypted metadata (#67). The correct transform is
```
encryptedMetadata = base64(string({ "iv": base64(iv), "ciphertext": base64(ciphertext) })
```
@mkleene @sujankota
fix(sdk): make sdk auto closeable (#63)
1d3aeee 
it contains resources that should be cleaned up when one is done with it
@mkleene @sujankota
fix(sdk): allow SDK to handle protocols in addresses (#70)
603edd6 
TDFs contain embedded URLs, some of which contain protocols. In order
for them to
work with GRPC we need to strip off the protocol.

The logic for ports is to use one if it is specified, otherwise we use
80 if the protocol is `http`,
otherwise use `443`.
@sujankota
fix the build
babb05e
@sujankota
merge main'
c7458be
@sujankota
fix the build
e5342ce
@sujankota sujankota merged commit 6485326 into main Jun 6, 2024
4 checks passed
@sujankota sujankota deleted the feature/nanotdf-encrypt branch June 6, 2024 18:37
@opentdf-automation opentdf-automation bot mentioned this pull request Jun 6, 2024
Merged
ttschampel added a commit to opentdf/nifi that referenced this pull request Jun 7, 2024
@ttschampel
feat: adding NanoTDF support (#5)
dfa5f52 
Adds support for NanoTDF: #3 

Pending upstream [java-sdk Nano
PR](opentdf/java-sdk#46)
ttschampel pushed a commit that referenced this pull request Jun 13, 2024
@opentdf-automation
chore(main): release 0.1.0 (#22)
56f0463 
🤖 I have created a release *beep* *boop*
---


<details><summary>0.1.0</summary>

## 0.1.0 (2024-06-13)


### Features

* add code to create services for SDK
([#35](#35))
([28513e6](28513e6))
* add logging ([#49](#49))
([9d20647](9d20647))
* Add NanoTDF E2E Tests
([#75](#75))
([84f9bd1](84f9bd1))
* **codegen:** Generate and publish Java Proto generated artifacts
([#2](#2))
([2328fd2](2328fd2))
* crypto API ([#33](#33))
([b8295b7](b8295b7))
* NanoTDF Implementation
([#46](#46))
([6485326](6485326))
* **PLAT-3087:** zip reader-writer
([#23](#23))
([3eeb626](3eeb626))
* SDK Encrypt (with mocked rewrap)
([#45](#45))
([d67daa2](d67daa2))
* **sdk:** add CLI and integration tests
([#64](#64))
([df20e6d](df20e6d))
* **sdk:** add ssl context
([#58](#58))
([80246a9](80246a9))
* **sdk:** get e2e rewrap working
([#52](#52))
([fe2c04b](fe2c04b))
* **sdk:** Issue [#60](#60) -
expose SDK ([#61](#61))
([ddef62a](ddef62a))
* **sdk:** provide access tokens dynamically to KAS
([#51](#51))
([04ca715](04ca715))
* **sdk:** update archive support
([#47](#47))
([29a80a9](29a80a9))


### Bug Fixes

* create TDFs larger than a single segment
([#65](#65))
([e1da325](e1da325))
* fix pom for release please
([#77](#77))
([3a3c357](3a3c357))
* Force BC provider use
([#76](#76))
([1bc9dd9](1bc9dd9))
* get rid of duplicate channel logic
([#59](#59))
([1edd666](1edd666))
* **sdk:** allow SDK to handle protocols in addresses
([#70](#70))
([97ae8ee](97ae8ee))
* **sdk:** make sdk auto closeable
([#63](#63))
([c1bbbb4](c1bbbb4))
</details>

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: opentdf-automation[bot] <149537512+opentdf-automation[bot]@users.noreply.github.com>
@ttschampel ttschampel mentioned this pull request Jun 18, 2024
Closed
2 tasks
This was referenced Aug 27, 2024
Closed
Closed
@opentdf-automation opentdf-automation bot mentioned this pull request Sep 13, 2024
Closed
@opentdf-automation opentdf-automation bot mentioned this pull request Oct 8, 2024
Merged
pflynn-virtru pushed a commit that referenced this pull request Oct 8, 2024
@opentdf-automation
chore(main): release 0.7.2 (#185)
c5c7116 
🤖 I have created a release *beep* *boop*
---


<details><summary>0.7.2</summary>

## [0.7.2](v0.7.0...v0.7.2)
(2024-10-08)


### ⚠ BREAKING CHANGES

* move to single jar
([#160](#160))

### Features

* add code to create services for SDK
([#35](#35))
([28513e6](28513e6))
* add logging ([#49](#49))
([9d20647](9d20647))
* Add NanoTDF E2E Tests
([#75](#75))
([84f9bd1](84f9bd1))
* adds token exchange and general auth
([#176](#176))
([bb325c4](bb325c4))
* BACK-2316 add a simple method to detect TDFs
([#111](#111))
([bfbef70](bfbef70))
* **build:** maven refactor for maven central
([#174](#174))
([c640773](c640773)),
closes [#79](#79)
* **ci:** Add xtest workflow trigger
([#96](#96))
([bc54b63](bc54b63))
* **cmd:** Adds command `--mime-type` opt
([#113](#113))
([45a2c30](45a2c30))
* **cmdline:** Adds --ecdsa-binding and help
([#164](#164))
([ed6e982](ed6e982))
* **codegen:** Generate and publish Java Proto generated artifacts
([#2](#2))
([2328fd2](2328fd2))
* **core:** Add attributes client
([#118](#118))
([98ba6a9](98ba6a9))
* **core:** Add autoconfigure for key splitting
([#120](#120))
([7ecbf23](7ecbf23))
* **core:** Adding key cache, tests for specificity
([#126](#126))
([a149887](a149887))
* **core:** Handle split keys on tdf3 encrypt and decrypt
([#109](#109))
([943751f](943751f))
* **core:** KID in NanoTDF
([#112](#112))
([33b5982](33b5982))
* **core:** NanoTDF resource locator protocol bit mask
([#107](#107))
([159d2f1](159d2f1))
* crypto API ([#33](#33))
([b8295b7](b8295b7))
* **lib:** add fallback to namespace kas
([#166](#166))
([4368840](4368840))
* NanoTDF Implementation
([#46](#46))
([6485326](6485326))
* **PLAT-3087:** zip reader-writer
([#23](#23))
([3eeb626](3eeb626))
* SDK Encrypt (with mocked rewrap)
([#45](#45))
([d67daa2](d67daa2))
* **sdk:** add CLI and integration tests
([#64](#64))
([df20e6d](df20e6d))
* **sdk:** add mime type.
([#108](#108))
([6c4a27b](6c4a27b))
* **sdk:** add ssl context
([#58](#58))
([80246a9](80246a9))
* **sdk:** expose GRPC auth service components
([#92](#92))
([2595cc5](2595cc5))
* **sdk:** get e2e rewrap working
([#52](#52))
([fe2c04b](fe2c04b))
* **sdk:** Issue [#60](#60) -
expose SDK ([#61](#61))
([ddef62a](ddef62a))
* **sdk:** provide access tokens dynamically to KAS
([#51](#51))
([04ca715](04ca715))
* **sdk:** the authorization service is needed for use by gateway
([#85](#85))
([73cac82](73cac82))
* **sdk:** update archive support
([#47](#47))
([29a80a9](29a80a9))
* **sdk:** Update the assertion support to match go sdk
([#117](#117))
([f9badb3](f9badb3))
* support key id in ztdf key access object
([#84](#84))
([862460a](862460a))
* update README.md
([#142](#142))
([198d335](198d335))


### Bug Fixes

* Align identifier bytes correctly in ResourceLocator
([#148](#148))
([2efe226](2efe226))
* **core:** Add support for certs
([#131](#131))
([2f98a3a](2f98a3a))
* **core:** Revert "feat(core): Add attributes client"
([#124](#124))
([3d1ef2b](3d1ef2b))
* create TDFs larger than a single segment
([#65](#65))
([e1da325](e1da325))
* fix pom for release please
([#77](#77))
([3a3c357](3a3c357))
* Force BC provider use
([#76](#76))
([1bc9dd9](1bc9dd9))
* get rid of duplicate channel logic
([#59](#59))
([1edd666](1edd666))
* GitHub packages snapshot repo
([#178](#178))
([713cb2b](713cb2b))
* GPG key and Maven credentials in release workflow
([#171](#171))
([864e9ce](864e9ce))
* Issue [#115](#115) - fix for
SSL Context for IDP and plaintext platform
([#116](#116))
([36a29df](36a29df))
* make sure we do not deserialize null
([#97](#97))
([9579c42](9579c42))
* **nano:** Store key ids if found
([#134](#134))
([94c672b](94c672b))
* passpharse ([#169](#169))
([8b3cbed](8b3cbed))
* policy-binding new structure
([#95](#95))
([b10a61e](b10a61e))
* **sdk:** allow SDK to handle protocols in addresses
([#70](#70))
([97ae8ee](97ae8ee))
* **sdk:** assertion support in tdf3
([#82](#82))
([c299dbd](c299dbd))
* **sdk:** give a test framework test scope
([#90](#90))
([b99de43](b99de43))
* **sdk:** make sdk auto closeable
([#63](#63))
([c1bbbb4](c1bbbb4))
* **sdk:** Mixed split fix
([#163](#163))
([649dac7](649dac7))
* ztdf support both base and handling assertions
([#128](#128))
([5f72e94](5f72e94))


### Documentation

* **sdk:** Adds brief usage code sample
([#26](#26))
([79215c7](79215c7))


### Miscellaneous Chores

* release 0.6.1 Release-As: 0.6.1
([#135](#135))
([09ec548](09ec548))
* release 0.7.2 ([#184](#184))
([ea6cf12](ea6cf12))


### Code Refactoring

* move to single jar
([#160](#160))
([ba9b2d5](ba9b2d5))
</details>

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: opentdf-automation[bot] <149537512+opentdf-automation[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ttschampel ttschampel ttschampel left review comments

@biscoe916 biscoe916 biscoe916 left review comments

@sujankota sujankota sujankota left review comments

@patmantru patmantru patmantru approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@mustyantsev @ttschampel @biscoe916 @sujankota @patmantru @mkleene