Skip to content

fix(core): add obligations X-Rewrap-Additional-Context to default CORS allowed headers [backport to release/service/v0.11] #2902

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
jakedoublev merged 1 commit into from
Nov 13, 2025
Merged

fix(core): add obligations X-Rewrap-Additional-Context to default CORS allowed headers [backport to release/service/v0.11] #2902

jakedoublev merged 1 commit into from
Nov 13, 2025

Conversation

@opentdf-automation
Copy link
Contributor

@opentdf-automation opentdf-automation bot commented Nov 13, 2025

Description

Backport of #2901 to release/service/v0.11.

@opentdf-automation opentdf-automation bot requested a review from a team as a code owner November 13, 2025 17:30
@opentdf-automation opentdf-automation bot mentioned this pull request Nov 13, 2025
Merged
@opentdf-automation
fix(core): add obligations X-Rewrap-Additional-Context to default COR…
10c4bee 
…S allowed headers (#2901)

### Proposed Changes

* This header is required for obligations and should be defaulted.
* Updates all example yamls with full list of default allowed headers.
* DSPX-1938

(cherry picked from commit d86868d)
@opentdf-automation opentdf-automation bot force-pushed the backport-2901-to-release/service/v0.11 branch from 09ea267 to 10c4bee Compare November 13, 2025 17:30
@github-actions
Copy link
Contributor

github-actions bot commented Nov 13, 2025

Benchmark results, click to expand

Benchmark authorization.GetDecisions Results:

Metric Value
Approved Decision Requests 1000
Denied Decision Requests 0
Total Time 185.18993ms

Benchmark authorization.v2.GetMultiResourceDecision Results:

Metric Value
Approved Decision Requests 1000
Denied Decision Requests 0
Total Time 86.107665ms

Benchmark Statistics

Name № Requests Avg Duration Min Duration Max Duration

Bulk Benchmark Results

Metric Value
Total Decrypts 100
Successful Decrypts 100
Failed Decrypts 0
Total Time 362.992472ms
Throughput 275.49 requests/second

TDF3 Benchmark Results:

Metric Value
Total Requests 5000
Successful Requests 5000
Failed Requests 0
Concurrent Requests 50
Total Time 39.010978918s
Average Latency 388.63666ms
Throughput 128.17 requests/second

NANOTDF Benchmark Results:

Metric Value
Total Requests 5000
Successful Requests 5000
Failed Requests 0
Concurrent Requests 50
Total Time 27.635603365s
Average Latency 275.494108ms
Throughput 180.93 requests/second

@github-actions
Copy link
Contributor

github-actions bot commented Nov 13, 2025

Benchmark results, click to expand

Benchmark authorization.GetDecisions Results:

Metric Value
Approved Decision Requests 1000
Denied Decision Requests 0
Total Time 173.262656ms

Benchmark authorization.v2.GetMultiResourceDecision Results:

Metric Value
Approved Decision Requests 1000
Denied Decision Requests 0
Total Time 102.078299ms

Benchmark Statistics

Name № Requests Avg Duration Min Duration Max Duration

Bulk Benchmark Results

Metric Value
Total Decrypts 100
Successful Decrypts 100
Failed Decrypts 0
Total Time 360.783457ms
Throughput 277.17 requests/second

TDF3 Benchmark Results:

Metric Value
Total Requests 5000
Successful Requests 5000
Failed Requests 0
Concurrent Requests 50
Total Time 39.756505625s
Average Latency 395.214084ms
Throughput 125.77 requests/second

NANOTDF Benchmark Results:

Metric Value
Total Requests 5000
Successful Requests 5000
Failed Requests 0
Concurrent Requests 50
Total Time 27.693014193s
Average Latency 275.970533ms
Throughput 180.55 requests/second

@github-actions
Copy link
Contributor

github-actions bot commented Nov 13, 2025

X-Test Results

opentdfplatformUQ2158.dockerbuild
cukes-report
✅ js-v0.4.34
✅ java-v0.4.34
✅ go-v0.4.34
bats-test-results
test-cases-mapping-report
✅ js-pull-2902
✅ java-pull-2902
✅ go-pull-2902

@github-actions
Copy link
Contributor

github-actions bot commented Nov 13, 2025

X-Test Results

opentdfplatformSGQL5D.dockerbuild
cukes-report
✅ java-v0.4.34
✅ js-v0.4.34
✅ go-v0.4.34
bats-test-results
test-cases-mapping-report
✅ js-09ea26705ffd0d98ac7007e4eba6a5989e3aae4f
✅ java-09ea26705ffd0d98ac7007e4eba6a5989e3aae4f
✅ go-09ea26705ffd0d98ac7007e4eba6a5989e3aae4f

@jakedoublev jakedoublev merged commit b9586b4 into release/service/v0.11 Nov 13, 2025
34 checks passed
@jakedoublev jakedoublev deleted the backport-2901-to-release/service/v0.11 branch November 13, 2025 17:48
@opentdf-automation opentdf-automation bot mentioned this pull request Nov 13, 2025
Merged
jakedoublev pushed a commit that referenced this pull request Nov 14, 2025
@opentdf-automation
chore(release): release service 0.11.4 (#2898)
7ebb535 
🤖 I have created a release *beep* *boop*
---


##
[0.11.4](service/v0.11.3...service/v0.11.4)
(2025-11-14)


### Features

* **kas:** add configurable SRT skew tolerance and diagnostics [backport
to release/service/v0.11]
([#2913](#2913))
([cac20cd](cac20cd))


### Bug Fixes

* **authz:** deny resources granularly when attribute value FQNs not
found [backport to release/service/v0.11]
([#2897](#2897))
([225627d](225627d))
* **core:** add obligations X-Rewrap-Additional-Context to default CORS
allowed headers [backport to release/service/v0.11]
([#2902](#2902))
([b9586b4](b9586b4))

---
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

@jakedoublev jakedoublev jakedoublev approved these changes

Assignees

No one assigned

Labels

size/s

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jakedoublev