Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

use a pre-generated test vectors to test hkdfExpandLabel #3175

Merged
merged 1 commit into from
May 29, 2021

Conversation

marten-seemann
Copy link
Member

The only reason we were using qtls.HkdfExpandLabel was to test our own
implementation of HKDF-Expand-Label. By using a pre-generated test
vector, we won't have to expose this function from qtls any more.

The only reason we were using qtls.HkdfExpandLabel was to test our own
implementation of HKDF-Expand-Label. By using a pre-generated test
vector, we won't have to expose this function from qtls any more.
@marten-seemann marten-seemann force-pushed the dont-use-qtls-hkdf-expand-label branch from 807384c to 05af55b Compare May 11, 2021 03:41
@codecov
Copy link

codecov bot commented May 11, 2021

Codecov Report

Merging #3175 (05af55b) into master (50746db) will increase coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3175      +/-   ##
==========================================
+ Coverage   85.35%   85.38%   +0.03%     
==========================================
  Files         133      133              
  Lines        9824     9822       -2     
==========================================
+ Hits         8385     8386       +1     
+ Misses       1063     1060       -3     
  Partials      376      376              
Impacted Files Coverage Δ
internal/qtls/go115.go 50.00% <ø> (+3.33%) ⬆️
internal/qtls/go116.go 50.00% <ø> (+3.33%) ⬆️
internal/utils/rand.go 75.00% <0.00%> (+12.50%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 50746db...05af55b. Read the comment docs.

@marten-seemann marten-seemann merged commit 0790066 into master May 29, 2021
@marten-seemann marten-seemann deleted the dont-use-qtls-hkdf-expand-label branch May 29, 2021 18:48
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants