Skip to content

fix(policy): fix casing bug and get provider config on update. [backport to release/service/v0.6] #2423

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
Open
wants to merge 1 commit into
base: release/service/v0.6
Choose a base branch
from
Open

fix(policy): fix casing bug and get provider config on update. [backport to release/service/v0.6] #2423

wants to merge 1 commit into from

Conversation

opentdf-automation[bot]
Copy link
Contributor

Description

Backport of #2403 to release/service/v0.6.

@opentdf-automation opentdf-automation bot requested review from a team as code owners June 9, 2025 21:54
@opentdf-automation opentdf-automation bot mentioned this pull request Jun 9, 2025
Merged
3 tasks
@opentdf-automation
fix(policy): fix casing bug and get provider config on update. (#2403)
823183c 
### Proposed Changes

1.) Update GetProviderConfig to lowercase the name
2.) Get the provider configuration after update.

### 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 a52b8f9)
@github-actions github-actions bot added the comp:db DB component label Jun 9, 2025
@opentdf-automation opentdf-automation bot force-pushed the backport-2403-to-release/service/v0.6 branch from 5815ae4 to 823183c Compare June 9, 2025 21:54
@github-actions github-actions bot added comp:policy Policy Configuration ( attributes, subject mappings, resource mappings, kas registry) size/s labels Jun 9, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jun 9, 2025

Benchmark results, click to expand

Benchmark authorization.GetDecisions Results:

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

Benchmark authorization.v2.GetMultiResourceDecision Results:

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

Bulk Benchmark Results

Metric Value
Total Decrypts 100
Successful Decrypts 100
Failed Decrypts 0
Total Time 344.154084ms
Throughput 290.57 requests/second

TDF3 Benchmark Results:

Metric Value
Total Requests 5000
Successful Requests 5000
Failed Requests 0
Concurrent Requests 50
Total Time 35.876871385s
Average Latency 356.926645ms
Throughput 139.37 requests/second

NANOTDF Benchmark Results:

Metric Value
Total Requests 5000
Successful Requests 5000
Failed Requests 0
Concurrent Requests 50
Total Time 25.343394465s
Average Latency 252.479907ms
Throughput 197.29 requests/second

Standard Benchmark Metrics Skipped or Failed

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jun 9, 2025

Benchmark results, click to expand

Benchmark authorization.GetDecisions Results:

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

Benchmark authorization.v2.GetMultiResourceDecision Results:

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

Bulk Benchmark Results

Metric Value
Total Decrypts 100
Successful Decrypts 100
Failed Decrypts 0
Total Time 353.83772ms
Throughput 282.62 requests/second

TDF3 Benchmark Results:

Metric Value
Total Requests 5000
Successful Requests 5000
Failed Requests 0
Concurrent Requests 50
Total Time 36.186345179s
Average Latency 359.650017ms
Throughput 138.17 requests/second

NANOTDF Benchmark Results:

Metric Value
Total Requests 5000
Successful Requests 5000
Failed Requests 0
Concurrent Requests 50
Total Time 25.376079215s
Average Latency 252.729063ms
Throughput 197.04 requests/second

Standard Benchmark Metrics Skipped or Failed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@strantalis strantalis strantalis approved these changes

Assignees
No one assigned
Labels
comp:db DB component comp:policy Policy Configuration ( attributes, subject mappings, resource mappings, kas registry) size/s
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@strantalis