This repository was archived by the owner on Nov 15, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
BLS Core Crypto attempt #2 #13618
Merged
Lederstrumpf
merged 49 commits into
paritytech:master
from
w3f:davxyn-skalman-core-bls-crypto
May 9, 2023
Merged
BLS Core Crypto attempt #2 #13618
Changes from 38 commits
Commits
Show all changes
49 commits
Select commit
Hold shift + click to select a range
b845210
Cherry pick all crypto related changes from pull-request #13311
drskalman 081ced0
Merge branch 'master' into davxyn-skalman-core-bls-crypto
davxy f411d74
Merge branch 'master' into davxyn-skalman-core-bls-crypto
davxy ee61ddb
Import some stuff just if 'full_crypto' is on
davxy a1e0f13
Remove copyright year
davxy 7023c14
Cleanup
davxy d92186d
First generic BLS draft
davxy 409b2f4
Finalize generic implementation
davxy 7177bb1
Restore tests
davxy 9b320cd
Merge pull request #13 from davxy/davxyn-skalman-core-bls-crypto-generic
davxy 58d4122
Fix rust docs
davxy 1e331c0
Merge branch 'master' into davxyn-skalman-core-bls-crypto
davxy a630ead
Fix after master merge
davxy 8a34867
Merge branch 'master' into davxyn-skalman-core-bls-crypto
davxy 5c47d33
Fix after master merge
davxy fe65fce
Use double bls with G1 as signature group and verify individual signa…
drskalman b278ba7
Fix inclusions and types used within substrate
davxy 3d50a72
Remove unused cruft
davxy 994a805
Restore usage of upstream crates
davxy d728537
Fix test
davxy fbb2f47
Reduce the diff by aligning Cargo.lock to master
davxy bc52143
Application-crypto provides bls381
davxy eb8a7ab
Merge branch 'master' into davxyn-skalman-core-bls-crypto
davxy 0acdf5c
Implement bls381 for local keystore
davxy 26d9ce2
Merge branch 'master' into davxyn-skalman-core-bls-crypto
davxy 9de1515
Use new generic keystore features
davxy 267538f
import DoublePublickey[Scheme] from the bls-like root to be less conf…
drskalman e3c2af8
Merge branch 'master' into davxyn-skalman-core-bls-crypto
andresilva 2a0a0ca
fix compilation
andresilva de48262
Apply suggestions from code review
davxy 7d709da
Clean leftovers
davxy a2b7308
- update bls test vector after applying spec change recommendation.
drskalman 478da9b
Merge branch 'davxyn-skalman-core-bls-crypto' of https://github.com/w…
drskalman 426c5ae
Different hard junction ids for different bls12 types
davxy da36af4
update to new bls-like
drskalman 856881a
Merge branch 'davxyn-skalman-core-bls-crypto' of https://github.com/w…
drskalman 0c7a5f7
bls-like → w3f-bls
drskalman 9218d9c
Make clippy happy
davxy 58c2d94
update test vector after replacing hash and crop with hash to field.
drskalman 6041fec
cargo fmt
drskalman f9b0da0
Merge branch 'master' into davxyn-skalman-core-bls-crypto
drskalman 758a169
Merge remote-tracking branch 'upstream/master' into HEAD
Lederstrumpf b11659d
account for #13972
Lederstrumpf 3a9c5ed
hide BLS behind "bls_non_production" feature flag
Lederstrumpf 0bfbcfd
Merge remote-tracking branch 'upstream/master' into davxyn-skalman-co…
Lederstrumpf 5a1a9cc
Remove Cargo.lock entries duplicated in merge
Lederstrumpf 2e48a99
add bls377 to primitives/keystore and client/keystore
drskalman 270b32e
rename feature `bls_non_production` to `bls-experimental`
drskalman 35b9167
Merge branch 'master' into davxyn-skalman-core-bls-crypto
drskalman File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,28 @@ | ||
| // This file is part of Substrate. | ||
|
|
||
| // Copyright (C) Parity Technologies (UK) Ltd. | ||
| // SPDX-License-Identifier: Apache-2.0 | ||
|
|
||
| // Licensed under the Apache License, Version 2.0 (the "License"); | ||
| // you may not use this file except in compliance with the License. | ||
| // You may obtain a copy of the License at | ||
| // | ||
| // http://www.apache.org/licenses/LICENSE-2.0 | ||
| // | ||
| // Unless required by applicable law or agreed to in writing, software | ||
| // distributed under the License is distributed on an "AS IS" BASIS, | ||
| // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
| // See the License for the specific language governing permissions and | ||
| // limitations under the License. | ||
|
|
||
| //! BLS12-381 crypto applications. | ||
|
|
||
| pub use sp_core::bls::bls381::*; | ||
|
|
||
| mod app { | ||
| crate::app_crypto!(super, sp_core::testing::BLS381); | ||
| } | ||
|
|
||
| #[cfg(feature = "full_crypto")] | ||
| pub use app::Pair as AppPair; | ||
| pub use app::{Public as AppPublic, Signature as AppSignature}; |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
at some point, we were discussing adding bls(381), but hiding it behind a feature flag (not
full_crypto, but a dedicated one to avoid production use). I may have missed some decisions/discussions in-between - you're now intentionally achieving the same by not implementingRuntimePublic?