Skip to content

Add ACVP Support for KTS-IFC#3009

Merged
nhatnghiho merged 6 commits intoaws:mainfrom
nhatnghiho:acvp-kts-ifc
Mar 9, 2026
Merged

Add ACVP Support for KTS-IFC#3009
nhatnghiho merged 6 commits intoaws:mainfrom
nhatnghiho:acvp-kts-ifc

Conversation

@nhatnghiho
Copy link
Copy Markdown
Contributor

Issues:

Addresses P355857148

Description of changes:

This PR adds ACVP support for KTS-IFC KTS-OAEP-basic

Call-outs:

Due to the random nature of RSA-OAEP encryption, no expected vector was added. To review, please run the script locally to verify the result.

Testing:

Run check_expected.go on the new KTS-IFC test vector.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and the ISC license.

Copy link
Copy Markdown
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

clang-tidy made some suggestions

Copy link
Copy Markdown
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

clang-tidy made some suggestions

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Feb 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.36%. Comparing base (110f184) to head (39d9551).
⚠️ Report is 76 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3009      +/-   ##
==========================================
+ Coverage   78.31%   78.36%   +0.04%     
==========================================
  Files         689      689              
  Lines      120995   121041      +46     
  Branches    16971    16969       -2     
==========================================
+ Hits        94758    94849      +91     
+ Misses      25341    25297      -44     
+ Partials      896      895       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

sgmenda
sgmenda previously approved these changes Feb 19, 2026
Copy link
Copy Markdown
Contributor

@sgmenda sgmenda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm with nits

Copy link
Copy Markdown
Contributor

@sgmenda sgmenda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@nhatnghiho nhatnghiho merged commit 343be17 into aws:main Mar 9, 2026
452 of 455 checks passed
WillChilds-Klein pushed a commit to WillChilds-Klein/aws-lc that referenced this pull request Mar 11, 2026
### Issues:
Addresses P355857148

### Description of changes: 
This PR adds ACVP support for `KTS-IFC KTS-OAEP-basic`

### Call-outs:
Due to the random nature of RSA-OAEP encryption, no expected vector was
added. To review, please run the script locally to verify the result.

### Testing:
Run `check_expected.go` on the new KTS-IFC test vector.

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license and the ISC license.
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.

4 participants