Skip to content
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

feat(sdk): Allow setting TDF mime type #797

Merged
merged 7 commits into from
May 16, 2024

Conversation

ntrevino-virtru
Copy link
Contributor

No description provided.

sdk/tdf.go Outdated Show resolved Hide resolved
@strantalis
Copy link
Member

@ntrevino-virtru

Originally when I wrote a go implementation I leveraged the http.DetectContentType function.

Wonder if we should pull that in and also allow an override.

https://github.com/strantalis/backend-go/blob/887ae6956966863af46d8288dc6906ac5a373466/pkg/tdf3/client/client.go#L178-L181

ntrevino-virtru and others added 2 commits May 15, 2024 18:38
Co-authored-by: Ryan Schumacher <jschumacher@virtru.com>
@ntrevino-virtru
Copy link
Contributor Author

@strantalis I'd personally vote no on that because that leaves room for unpredictability which I'm not a fan of. I'm just me though, so happy to if the general consensus is in favor.

@ntrevino-virtru ntrevino-virtru force-pushed the feature/ccr-2825/set-tdf-mime-type branch 2 times, most recently from f363a8f to 289f881 Compare May 16, 2024 01:42
@ntrevino-virtru ntrevino-virtru marked this pull request as ready for review May 16, 2024 03:36
@ntrevino-virtru ntrevino-virtru requested review from a team as code owners May 16, 2024 03:36
Can't seem to get it to run locally, so running on CI :)
@ntrevino-virtru ntrevino-virtru force-pushed the feature/ccr-2825/set-tdf-mime-type branch from 289f881 to 66bc4b6 Compare May 16, 2024 19:40
@dmihalcik-virtru dmihalcik-virtru added this pull request to the merge queue May 16, 2024
Merged via the queue into main with commit 97926a1 May 16, 2024
16 checks passed
@dmihalcik-virtru dmihalcik-virtru deleted the feature/ccr-2825/set-tdf-mime-type branch May 16, 2024 21:10
github-merge-queue bot pushed a commit that referenced this pull request May 21, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.2.3](sdk/v0.2.2...sdk/v0.2.3)
(2024-05-21)


### Features

* **authz:** Handle jwts as entity chains in decision requests
([#759](#759))
([65612e0](65612e0))
* **sdk:** Allow setting TDF mime type
([#797](#797))
([97926a1](97926a1))


### Bug Fixes

* bump internal versions
([#840](#840))
([8f45f18](8f45f18))

---
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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants