Aleksandr Bezobchuk
cfd42be0fe
Improve error and tests
6 years ago
Alessio Treglia
7644d27307
Ensure multisig keys have 20-byte address ( #3103 )
* Ensure multisig keys have 20-byte address
Use crypto.AddressHash() to avoid returning 32-byte long address.
Closes : #3102
* fix pointer
* fix test
6 years ago
Alessio Treglia
764cfe33aa
Don't use pointer receivers for PubKeyMultisigThreshold ( #3100 )
* Don't use pointer receivers for PubKeyMultisigThreshold
* test that showcases panic when PubKeyMultisigThreshold are used in sdk:
- deserialization will fail in `readInfo` which tries to read a
`crypto.PubKey` into a `localInfo` (called by
cosmos-sdk/client/keys.GetKeyInfo)
* Update changelog
* Rename routeTable to nameTable, multisig key is no longer a pointer
* sed -i 's/PubKeyAminoRoute/PubKeyAminoName/g' `grep -lrw PubKeyAminoRoute .`
upon Jae's request
* AminoRoutes -> AminoNames
* sed -e 's/PrivKeyAminoRoute/PrivKeyAminoName/g'
* Update crypto/encoding/amino/amino.go
Co-Authored-By: alessio <quadrispro@ubuntu.com>
6 years ago
Ethan Buchman
38b401657e
Cleanup up Multisig naming ( #2255 )
* crypto/multisig: Pubkey -> PubKey
* crypto/encoding/amino: use pkg vars for routes
* crypto/multisig/bitarray
* crypto/multisig: ThresholdMultiSignaturePubKey -> PubKeyMultisigThreshold
* crypto/encoding/amino: add PubKeyMultisig to table
* remove bA import alias
https://github.com/tendermint/tendermint/pull/2255#discussion_r211900709
6 years ago
Dev Ojha
728d2ed266
crypto: Remove unnecessary prefixes from amino route variable names ( #2205 )
* crypto: Remove unnecessary ed25519 and secp256k1 prefixes from amino routes.
* (squash this) add changelog
* (squash this) multisig amino fixes
* (squash this) fix build error
6 years ago
ValarDragon
2fe34491ba
(squash this) Fix build errors
6 years ago
ValarDragon
6beaf6e72d
(squash this) address Jae's comments on `NumTrueBitsBefore`
6 years ago
ValarDragon
8d28344e84
(Squash this) switch to bare
6 years ago
ValarDragon
4cf1dbd676
(squash this) fix amino route
6 years ago
ValarDragon
00db469fc0
(squash this) begin addressing PR comments
6 years ago
Dev Ojha
5a8fe61200
crypto: Add compact bit array for intended usage in the multisig ( #2140 )
* crypto: Add compact bit array for intended usage in the multisig
This is in a separate PR for ease of review.
* (squash this) add comment
6 years ago
ValarDragon
4e7bf10b59
(squash this) squashed bug with multiple signatures at same index.
6 years ago
ValarDragon
67b6d51ff4
(squash this) address PR comments + fix bug in equality check
6 years ago
ValarDragon
e7dd76c28d
crypto: Threshold multisig implementation
6 years ago
ValarDragon
21448bcf4f
crypto: Add compact bit array for intended usage in the multisig
This is in a separate PR for ease of review.
6 years ago
ValarDragon
96fdec0fca
crypto: Add compact bit array for intended usage in the multisig
This is in a separate PR for ease of review.
6 years ago