Skip to content

Conversation

@opentdf-automation
Copy link
Contributor

Description

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

@opentdf-automation opentdf-automation bot requested review from a team as code owners November 3, 2025 17:54
### Proposed Changes

1.) Populate kas rewrap audit message with attributes.
2.) Add keyID to `eventmetadata`

#### Example Nano failure
```json
{
    "time": "2025-10-30T10:13:30.270403-05:00",
    "level": "AUDIT",
    "msg": "rewrap",
    "namespace": "kas",
    "audit": {
        "object": {
            "type": "key_object",
            "id": "ff1a2fe2-a942-11f0-9751-a6a754e79d24",
            "name": "",
            "attributes": {
                "assertions": [],
                "attrs": [
                    "https://test.obligations/attr/test_attr_for_triggers/value/test_valu_for_trigger"
                ],
                "permissions": []
            }
        },
        "action": {
            "type": "rewrap",
            "result": "error"
        },
        "actor": {
            "id": "031fe452-ddbb-4d36-b82f-c6b3dd4d122a",
            "attributes": []
        },
        "eventMetaData": {
            "algorithm": "ec:secp256r1",
            "keyID": "e1",
            "policyBinding": "",
            "tdfFormat": "Nano"
        },
        "clientInfo": {
            "userAgent": "connect-go/1.18.1 (go1.24.6)",
            "platform": "kas",
            "requestIP": "None"
        },
        "original": null,
        "updated": null,
        "requestID": "d56da397-3387-4a14-9955-73681e627e37",
        "timestamp": "2025-10-30T10:13:30-05:00"
    }
}
```

#### Example ztdf success
```json
{
    "time": "2025-10-31T11:58:40.892713-05:00",
    "level": "AUDIT",
    "msg": "rewrap",
    "namespace": "kas",
    "audit": {
        "object": {
            "type": "key_object",
            "id": "cd2f0354-a942-11f0-b197-a6a754e79d24",
            "name": "",
            "attributes": {
                "assertions": [],
                "attrs": [
                    "https://test.obligations/attr/test_attr_for_triggers/value/test_valu_for_trigger"
                ],
                "permissions": []
            }
        },
        "action": {
            "type": "rewrap",
            "result": "success"
        },
        "actor": {
            "id": "031fe452-ddbb-4d36-b82f-c6b3dd4d122a",
            "attributes": []
        },
        "eventMetaData": {
            "algorithm": "rsa:2048",
            "keyID": "r1",
            "policyBinding": "YjEwNWMwZGVhMjkzYjBhZjU4MWNkOTE1MmU4N2NkNjkzNzQ2ODM5NDI0MGRjYjhmNjRiZjlhNmY0OWEzZjJlNw==",
            "tdfFormat": "tdf3"
        },
        "clientInfo": {
            "userAgent": "connect-go/1.18.1 (go1.24.6)",
            "platform": "kas",
            "requestIP": "None"
        },
        "original": null,
        "updated": null,
        "requestID": "73f131ae-cc21-490c-868d-260e58b8664d",
        "timestamp": "2025-10-31T11:58:40-05:00"
    }
}
```

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

(cherry picked from commit 4fe97fd)
@opentdf-automation opentdf-automation bot force-pushed the backport-2861-to-release/service/v0.11 branch from b03d861 to 874e693 Compare November 3, 2025 17:54
@github-actions github-actions bot added comp:kas Key Access Server size/s labels Nov 3, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2025

Benchmark results, click to expand

Benchmark authorization.GetDecisions Results:

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

Benchmark authorization.v2.GetMultiResourceDecision Results:

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

Standard Benchmark Metrics Skipped or Failed

Bulk Benchmark Results

Metric Value
Total Decrypts 100
Successful Decrypts 100
Failed Decrypts 0
Total Time 360.350315ms
Throughput 277.51 requests/second

TDF3 Benchmark Results:

Metric Value
Total Requests 5000
Successful Requests 5000
Failed Requests 0
Concurrent Requests 50
Total Time 38.330352902s
Average Latency 381.009138ms
Throughput 130.44 requests/second

NANOTDF Benchmark Results:

Metric Value
Total Requests 5000
Successful Requests 5000
Failed Requests 0
Concurrent Requests 50
Total Time 27.216307152s
Average Latency 271.265976ms
Throughput 183.71 requests/second

@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2025

Benchmark results, click to expand

Benchmark authorization.GetDecisions Results:

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

Benchmark authorization.v2.GetMultiResourceDecision Results:

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

Standard Benchmark Metrics Skipped or Failed

Bulk Benchmark Results

Metric Value
Total Decrypts 100
Successful Decrypts 100
Failed Decrypts 0
Total Time 410.762929ms
Throughput 243.45 requests/second

TDF3 Benchmark Results:

Metric Value
Total Requests 5000
Successful Requests 5000
Failed Requests 0
Concurrent Requests 50
Total Time 42.373269206s
Average Latency 421.574707ms
Throughput 118.00 requests/second

NANOTDF Benchmark Results:

Metric Value
Total Requests 5000
Successful Requests 5000
Failed Requests 0
Concurrent Requests 50
Total Time 28.553333035s
Average Latency 284.605099ms
Throughput 175.11 requests/second

@c-r33d c-r33d merged commit f460972 into release/service/v0.11 Nov 3, 2025
34 checks passed
@c-r33d c-r33d deleted the backport-2861-to-release/service/v0.11 branch November 3, 2025 18:36
jakedoublev pushed a commit that referenced this pull request Nov 7, 2025
🤖 I have created a release *beep* *boop*
---


##
[0.11.3](service/v0.11.2...service/v0.11.3)
(2025-11-07)


### Features

* **kas:** Add nano policy binding to rewrap audit. [backport to
release/service/v0.11]
([#2880](#2880))
([d8558f3](d8558f3))


### Bug Fixes

* **authz:** if entity identifier results in multiple representations,
treat with AND in resource decision results [backport to
release/service/v0.11]
([#2887](#2887))
([f772064](f772064))
* **kas:** populate rewrap audit log [backport to release/service/v0.11]
([#2867](#2867))
([f460972](f460972))

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

comp:kas Key Access Server size/s

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants