Skip to content

StaticKeyPair in agreement module#1081

Open
richard-w wants to merge 3 commits intobriansmith:mainfrom
richard-w:feature/agreement-static-key
Open

StaticKeyPair in agreement module#1081
richard-w wants to merge 3 commits intobriansmith:mainfrom
richard-w:feature/agreement-static-key

Conversation

@richard-w
Copy link
Copy Markdown

Adds agreement::StaticKeyPair and agreement::agree_static

Also restructures some existing tests to avoid duplication of code and test vectors.

Closes: #331

`agreement_agree_ephemeral` still tests the ephemeral key agreement. The
public key tests have been moved to `agreement_public_key`.
In contrast to `EphemeralPrivateKey` a `StaticKeyPair` can be used
multiple times.
@richard-w
Copy link
Copy Markdown
Author

The test failure is a spurious error in appveyor.

@richard-w richard-w force-pushed the feature/agreement-static-key branch from 453c6cc to 5282885 Compare November 10, 2020 19:01
@richard-w richard-w force-pushed the feature/agreement-static-key branch from 97eb7f5 to d151810 Compare December 20, 2020 09:47
@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 20, 2020

Codecov Report

Merging #1081 (0da2432) into main (27200d4) will increase coverage by 0.03%.
The diff coverage is 96.21%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1081      +/-   ##
==========================================
+ Coverage   91.97%   92.01%   +0.03%     
==========================================
  Files         106      106              
  Lines       14238    14395     +157     
==========================================
+ Hits        13096    13246     +150     
- Misses       1142     1149       +7     
Impacted Files Coverage Δ
tests/agreement_tests.rs 95.51% <94.00%> (-1.80%) ⬇️
src/agreement.rs 88.00% <98.82%> (+2.65%) ⬆️
src/aead/aes.rs 75.23% <0.00%> (-0.48%) ⬇️
src/ec/curve25519/x25519.rs 98.86% <0.00%> (+0.05%) ⬆️
tests/aead_tests.rs 97.13% <0.00%> (+0.35%) ⬆️
src/ec/keys.rs 98.21% <0.00%> (+1.78%) ⬆️
src/hkdf.rs 88.50% <0.00%> (+4.81%) ⬆️

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 27200d4...0da2432. Read the comment docs.

@richard-w richard-w force-pushed the feature/agreement-static-key branch from d151810 to 0da2432 Compare December 20, 2020 11:35
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.

X25519 (and/or ECDH in general) with static keys

1 participant