Skip to content

Conversation

jakedoublev
Copy link
Contributor

@jakedoublev jakedoublev commented Sep 18, 2025

Proposed Changes

  • Adds fulfillable obligations to request protos
  • Adds required obligations to each ResourceDecision in protos
  • FQN-driven (not UUIDs)
  • protovalidate rules allow up to 50 fulfillable obligations in a request

Checklist

  • I have added or updated unit tests
  • I have added or updated integration tests (if appropriate)
  • I have added or updated documentation

Testing Instructions

@jakedoublev jakedoublev marked this pull request as ready for review September 18, 2025 19:53
@jakedoublev jakedoublev requested review from a team as code owners September 18, 2025 19:53
Copy link
Contributor

Benchmark results, click to expand

Benchmark authorization.GetDecisions Results:

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

Benchmark authorization.v2.GetMultiResourceDecision Results:

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

Standard Benchmark Metrics Skipped or Failed

Bulk Benchmark Results

Metric Value
Total Decrypts 100
Successful Decrypts 100
Failed Decrypts 0
Total Time 353.915052ms
Throughput 282.55 requests/second

TDF3 Benchmark Results:

Metric Value
Total Requests 5000
Successful Requests 5000
Failed Requests 0
Concurrent Requests 50
Total Time 36.816517229s
Average Latency 367.105408ms
Throughput 135.81 requests/second

NANOTDF Benchmark Results:

Metric Value
Total Requests 5000
Successful Requests 5000
Failed Requests 0
Concurrent Requests 50
Total Time 25.58147361s
Average Latency 254.653204ms
Throughput 195.45 requests/second

@jakedoublev jakedoublev added this pull request to the merge queue Sep 18, 2025
Merged via the queue into main with commit 41ee5a8 Sep 18, 2025
36 checks passed
@jakedoublev jakedoublev deleted the feat/DSPX-1355 branch September 18, 2025 20:28
github-merge-queue bot pushed a commit that referenced this pull request Sep 18, 2025
🤖 I have created a release *beep* *boop*
---


##
[0.11.0](protocol/go/v0.10.0...protocol/go/v0.11.0)
(2025-09-18)


### Features

* **authz:** obligations protos within auth service
([#2745](#2745))
([41ee5a8](41ee5a8))
* **policy:** Return obligations from GetAttributeValue calls
([#2742](#2742))
([aa9b393](aa9b393))

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

Successfully merging this pull request may close these issues.

2 participants