Skip to content

fix(sdk): Interpret old rewrap errors #2013

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.

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

dmihalcik-virtru
Copy link
Member

  • Older server rewrap errors would be miscategorized

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

- Older server rewrap errors would be miscategorized
@dmihalcik-virtru dmihalcik-virtru marked this pull request as ready for review March 31, 2025 17:47
@dmihalcik-virtru dmihalcik-virtru requested review from a team as code owners March 31, 2025 17:47
Copy link
Contributor

Benchmark Statistics

Name № Requests Avg Duration Min Duration Max Duration
publickey 3 0.082 ms 0.070 ms 0.093 ms
rewrap-nanotdf 4710 438.050 ms 61.021 ms 1106.944 ms
rewrap-tdf3 5001 496.664 ms 64.459 ms 1196.058 ms

Bulk Benchmark Results

Metric Value
Total Decrypts 100
Successful Decrypts 100
Failed Decrypts 0
Total Time 386.29558ms
Throughput 258.87 requests/second

Copy link
Contributor

Benchmark Statistics

Name № Requests Avg Duration Min Duration Max Duration
publickey 3 0.063 ms 0.040 ms 0.094 ms
rewrap-nanotdf 4966 413.012 ms 18.306 ms 939.299 ms
rewrap-tdf3 5001 441.428 ms 51.821 ms 1026.128 ms

Bulk Benchmark Results

Metric Value
Total Decrypts 100
Successful Decrypts 100
Failed Decrypts 0
Total Time 355.126488ms
Throughput 281.59 requests/second

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant