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

evm: add curve25519 public key generation precompile #1268

Merged
merged 1 commit into from
Feb 22, 2023

Conversation

nhynes
Copy link
Contributor

@nhynes nhynes commented Feb 20, 2023

This PR adds a curve25519 public key generation precompile. This was added because it's not totally cool to use ed25519 keys for DH. Goes with oasisprotocol/sapphire-paratime#116.

@codecov
Copy link

codecov bot commented Feb 21, 2023

Codecov Report

Merging #1268 (e778f0e) into main (d3ce6d3) will increase coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1268      +/-   ##
==========================================
+ Coverage   67.97%   68.02%   +0.05%     
==========================================
  Files         133      133              
  Lines       12549    12571      +22     
==========================================
+ Hits         8530     8552      +22     
  Misses       3989     3989              
  Partials       30       30              
Impacted Files Coverage Δ
...ime-sdk/modules/evm/src/precompile/confidential.rs 92.82% <100.00%> (+0.33%) ⬆️
runtime-sdk/modules/evm/src/precompile/mod.rs 95.23% <100.00%> (+0.11%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@nhynes nhynes force-pushed the nhynes/curve25519-precompile branch from 31fc1fb to e778f0e Compare February 21, 2023 01:34
kostko
kostko approved these changes Feb 21, 2023
@nhynes nhynes merged commit e49f5a5 into main Feb 22, 2023
@nhynes nhynes deleted the nhynes/curve25519-precompile branch February 22, 2023 19:00
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.

2 participants