Skip to content
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

AES Key Wrap Algorithm Support #250

Merged
merged 3 commits into from
Jan 8, 2024
Merged

AES Key Wrap Algorithm Support #250

merged 3 commits into from
Jan 8, 2024

Conversation

skmcgrail
Copy link
Member

Description of changes:

Adds aws_lc_rs::key_wrap module that provides AES-128 and AES-256 key wrap algorithms (padded and unpadded variants).

Testing:

Pulled a partial set of key wrap tests from the available CAVP tests.

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.

@skmcgrail skmcgrail requested a review from a team as a code owner October 20, 2023 16:33
aws-lc-rs/src/key_wrap.rs Outdated Show resolved Hide resolved
aws-lc-rs/src/key_wrap.rs Outdated Show resolved Hide resolved
@codecov-commenter
Copy link

codecov-commenter commented Oct 24, 2023

Codecov Report

Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (fa6733a) 95.21% compared to head (50a0c43) 95.66%.
Report is 6 commits behind head on main.

Files Patch % Lines
aws-lc-rs/src/key_wrap.rs 97.25% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #250      +/-   ##
==========================================
+ Coverage   95.21%   95.66%   +0.45%     
==========================================
  Files          55       57       +2     
  Lines        6980     7751     +771     
==========================================
+ Hits         6646     7415     +769     
- Misses        334      336       +2     

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

@skmcgrail skmcgrail force-pushed the keywrap branch 2 times, most recently from 651c693 to 421497e Compare October 24, 2023 16:53
aws-lc-rs/src/key_wrap.rs Outdated Show resolved Hide resolved
aws-lc-rs/src/key_wrap.rs Outdated Show resolved Hide resolved
aws-lc-rs/src/key_wrap.rs Outdated Show resolved Hide resolved
aws-lc-rs/src/key_wrap.rs Outdated Show resolved Hide resolved
aws-lc-rs/src/key_wrap.rs Outdated Show resolved Hide resolved
aws-lc-rs/src/key_wrap.rs Outdated Show resolved Hide resolved
Copy link
Contributor

@justsmth justsmth left a comment

Choose a reason for hiding this comment

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

Looks good overall. I have a couple of questions we should consider before we finalize this as our "key wrap" API.

aws-lc-rs/src/key_wrap.rs Outdated Show resolved Hide resolved
aws-lc-rs/src/key_wrap.rs Outdated Show resolved Hide resolved
aws-lc-rs/src/key_wrap.rs Outdated Show resolved Hide resolved
justsmth
justsmth previously approved these changes Dec 13, 2023
@skmcgrail skmcgrail requested review from WillChilds-Klein and removed request for samuel40791765 December 14, 2023 18:09
aws-lc-rs/src/key_wrap.rs Show resolved Hide resolved
aws-lc-rs/src/key_wrap.rs Show resolved Hide resolved
aws-lc-rs/src/key_wrap.rs Show resolved Hide resolved
aws-lc-rs/src/key_wrap.rs Show resolved Hide resolved
aws-lc-rs/src/key_wrap.rs Show resolved Hide resolved
aws-lc-rs/src/key_wrap.rs Show resolved Hide resolved
aws-lc-rs/src/key_wrap.rs Show resolved Hide resolved
aws-lc-rs/src/key_wrap.rs Show resolved Hide resolved
aws-lc-rs/src/key_wrap/tests/fips.rs Show resolved Hide resolved
aws-lc-rs/src/key_wrap/tests.rs Outdated Show resolved Hide resolved
@skmcgrail skmcgrail merged commit f3750b6 into aws:main Jan 8, 2024
103 of 109 checks passed
@skmcgrail skmcgrail deleted the keywrap branch January 8, 2024 17:33
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