Skip to content

fix(core): Allow 521 curve to be used [backport to release/protocol/go/v0.5] #2489

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 1 commit into from
Jun 26, 2025
Merged

fix(core): Allow 521 curve to be used [backport to release/protocol/go/v0.5] #2489

merged 1 commit into from
Jun 26, 2025

Conversation

@opentdf-automation
Copy link
Contributor

@opentdf-automation opentdf-automation bot commented Jun 26, 2025

Description

Backport of #2485 to release/protocol/go/v0.5.

@opentdf-automation opentdf-automation bot requested review from a team as code owners June 26, 2025 19:26
@opentdf-automation opentdf-automation bot mentioned this pull request Jun 26, 2025
Merged
3 tasks
@opentdf-automation
fix(core): Allow 521 curve to be used (#2485)
7bf86ea 
### Proposed Changes

*

### 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 aaf43dc)
@opentdf-automation opentdf-automation bot force-pushed the backport-2485-to-release/protocol/go/v0.5 branch from 2c5636d to 7bf86ea Compare June 26, 2025 19:26
@github-actions github-actions bot added comp:policy Policy Configuration ( attributes, subject mappings, resource mappings, kas registry) docs Documentation size/xs labels Jun 26, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Jun 26, 2025

Benchmark results, click to expand

Benchmark authorization.GetDecisions Results:

Metric Value
Approved Decision Requests 5000
Denied Decision Requests 0
Total Time 532.307754ms

Benchmark authorization.v2.GetMultiResourceDecision Results:

Metric Value
Approved Decision Requests 5000
Denied Decision Requests 0
Total Time 338.441661ms

Standard Benchmark Metrics Skipped or Failed

Bulk Benchmark Results

Metric Value
Total Decrypts 100
Successful Decrypts 100
Failed Decrypts 0
Total Time 563.353529ms
Throughput 177.51 requests/second

TDF3 Benchmark Results:

Metric Value
Total Requests 5000
Successful Requests 5000
Failed Requests 0
Concurrent Requests 50
Total Time 36.192328305s
Average Latency 360.250971ms
Throughput 138.15 requests/second

NANOTDF Benchmark Results:

Metric Value
Total Requests 5000
Successful Requests 5000
Failed Requests 0
Concurrent Requests 50
Total Time 25.305332736s
Average Latency 252.159695ms
Throughput 197.59 requests/second

@github-actions
Copy link
Contributor

github-actions bot commented Jun 26, 2025

Benchmark results, click to expand

Benchmark authorization.GetDecisions Results:

Metric Value
Approved Decision Requests 5000
Denied Decision Requests 0
Total Time 526.05499ms

Benchmark authorization.v2.GetMultiResourceDecision Results:

Metric Value
Approved Decision Requests 5000
Denied Decision Requests 0
Total Time 351.23936ms

Standard Benchmark Metrics Skipped or Failed

Bulk Benchmark Results

Metric Value
Total Decrypts 100
Successful Decrypts 100
Failed Decrypts 0
Total Time 530.41797ms
Throughput 188.53 requests/second

TDF3 Benchmark Results:

Metric Value
Total Requests 5000
Successful Requests 5000
Failed Requests 0
Concurrent Requests 50
Total Time 36.342449524s
Average Latency 361.083571ms
Throughput 137.58 requests/second

NANOTDF Benchmark Results:

Metric Value
Total Requests 5000
Successful Requests 5000
Failed Requests 0
Concurrent Requests 50
Total Time 25.785973288s
Average Latency 256.601072ms
Throughput 193.90 requests/second

@dmihalcik-virtru dmihalcik-virtru merged commit 403409c into release/protocol/go/v0.5 Jun 26, 2025
25 checks passed
@dmihalcik-virtru dmihalcik-virtru deleted the backport-2485-to-release/protocol/go/v0.5 branch June 26, 2025 20:11
@opentdf-automation opentdf-automation bot mentioned this pull request Jun 26, 2025
Merged
dmihalcik-virtru pushed a commit that referenced this pull request Jun 30, 2025
@opentdf-automation
chore(ci): release protocol/go 0.5.1 (#2490)
edc98ad 
🤖 I have created a release *beep* *boop*
---


##
[0.5.1](protocol/go/v0.5.0...protocol/go/v0.5.1)
(2025-06-26)


### Bug Fixes

* **core:** Allow 521 curve to be used [backport to
release/protocol/go/v0.5]
([#2489](#2489))
([403409c](403409c))

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

@dmihalcik-virtru dmihalcik-virtru dmihalcik-virtru approved these changes

@c-r33d c-r33d c-r33d approved these changes

Assignees

No one assigned

Labels

comp:policy Policy Configuration ( attributes, subject mappings, resource mappings, kas registry) docs Documentation size/xs

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@dmihalcik-virtru @c-r33d