Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

swap ed25519-dalek for ed25519-zebra #11781

Merged
merged 13 commits into from
Aug 15, 2022

Conversation

jakehemmerle
Copy link
Contributor

Please see #8764 as it explains all of this.

Fixes #8055

Polkadot burn in: paritytech/polkadot#5747

jakehemmerle and others added 2 commits June 30, 2022 18:04

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
fixed batch verificaiton tests

removed additional zero verificaiton tests

removed comments, fixed test bug, added #[derive(Clone)]

Update primitives/core/src/ed25519.rs

Co-authored-by: Squirrel <gilescope@gmail.com>
@jakehemmerle jakehemmerle changed the title 8805 ed25519 zebra swap ed25519-dalek for ed25519-zebra Jul 5, 2022
@jakehemmerle
Copy link
Contributor Author

@bkchr fixed all cicd; should be ready

@bkchr bkchr requested review from koute and davxy August 12, 2022 19:51
@bkchr bkchr added A0-please_review Pull request needs code review. B5-clientnoteworthy C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit labels Aug 12, 2022
Co-authored-by: Bastian Köcher <bkchr@users.noreply.github.com>
Copy link
Member

@davxy davxy left a comment

Choose a reason for hiding this comment

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

LGTM.

But please update Cargo.lock first

jakehemmerle and others added 6 commits August 13, 2022 13:42
Co-authored-by: Davide Galassi <davxy@datawok.net>
Co-authored-by: Davide Galassi <davxy@datawok.net>
Co-authored-by: Davide Galassi <davxy@datawok.net>
Co-authored-by: Davide Galassi <davxy@datawok.net>
@jakehemmerle jakehemmerle requested review from davxy and bkchr August 13, 2022 17:55
Copy link
Contributor

@koute koute left a comment

Choose a reason for hiding this comment

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

From what I can see non-batched performance seems to be unchanged compared to ed25519-dalek, so LGTM.

@bkchr bkchr merged commit cf36d2f into paritytech:master Aug 15, 2022
ark0f pushed a commit to gear-tech/substrate that referenced this pull request Feb 27, 2023
* swap ed25519-dalek for ed25519-zebra; no batch verificaiton

fixed batch verificaiton tests

removed additional zero verificaiton tests

removed comments, fixed test bug, added #[derive(Clone)]

Update primitives/core/src/ed25519.rs

Co-authored-by: Squirrel <gilescope@gmail.com>

* modified assertion to allow default ed25519-zebra zero key behavior

* cargo clippy

* Update primitives/core/Cargo.toml

Co-authored-by: Bastian Köcher <bkchr@users.noreply.github.com>

* Update primitives/core/src/ed25519.rs

Co-authored-by: Davide Galassi <davxy@datawok.net>

* Update primitives/core/src/ed25519.rs

Co-authored-by: Davide Galassi <davxy@datawok.net>

* Update primitives/core/src/ed25519.rs

Co-authored-by: Davide Galassi <davxy@datawok.net>

* Update primitives/core/src/ed25519.rs

Co-authored-by: Davide Galassi <davxy@datawok.net>

* updated Cargo.lock for sp-core

* fix inaccurate comment

Co-authored-by: Squirrel <gilescope@gmail.com>
Co-authored-by: Bastian Köcher <info@kchr.de>
Co-authored-by: Bastian Köcher <bkchr@users.noreply.github.com>
Co-authored-by: Davide Galassi <davxy@datawok.net>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ed25519-zebra
4 participants