-
Notifications
You must be signed in to change notification settings - Fork 49
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
Remove arbitrary HKDF info length limit #424
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the super quick turnaround! The patch looks great to me. I also switched my downstream branch to use this and was able to run through the problematic HPKE test vectors without error 👍
1e5bc72
to
aa940f7
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #424 +/- ##
==========================================
- Coverage 95.80% 92.67% -3.13%
==========================================
Files 61 61
Lines 8143 8174 +31
Branches 0 8174 +8174
==========================================
- Hits 7801 7575 -226
- Misses 342 357 +15
- Partials 0 242 +242 ☔ View full report in Codecov by Sentry. |
e1762a8
to
fed7282
Compare
Description of changes:
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.