Browse Source

update go-amino

pull/1782/head
Anton Kaliaev 7 years ago
parent
commit
f499ce8713
No known key found for this signature in database GPG Key ID: 7B6881D965918214
10 changed files with 44 additions and 43 deletions
  1. +3
    -3
      Gopkg.lock
  2. +1
    -1
      Gopkg.toml
  3. +1
    -1
      README.md
  4. +4
    -4
      _nano/keys.go
  5. +11
    -11
      amino.go
  6. +8
    -8
      encode_test.go
  7. +1
    -1
      glide.lock
  8. +1
    -1
      glide.yaml
  9. +4
    -4
      keys/wire.go
  10. +10
    -9
      signature_test.go

+ 3
- 3
Gopkg.lock View File

@ -125,9 +125,9 @@
[[projects]]
branch = "develop"
name = "github.com/tendermint/go-wire"
name = "github.com/tendermint/go-amino"
packages = ["."]
revision = "dec83f641903b22f039da3974607859715d0377e"
revision = "3b9e2b978447707c255922bc3f87a53d55c400c9"
[[projects]]
branch = "develop"
@ -163,6 +163,6 @@
[solve-meta]
analyzer-name = "dep"
analyzer-version = 1
inputs-digest = "e0628df240b8ceeb91403f5218f5561d8580f15f3d5b0ea0da40710d1cba3707"
inputs-digest = "375b661ad202b62c6847981416c03ce0518c33ac293f5f0863b69af04d2af91f"
solver-name = "gps-cdcl"
solver-version = 1

+ 1
- 1
Gopkg.toml View File

@ -47,7 +47,7 @@
[[constraint]]
branch = "develop"
name = "github.com/tendermint/go-wire"
name = "github.com/tendermint/go-amino"
[[constraint]]
branch = "develop"


+ 1
- 1
README.md View File

@ -3,4 +3,4 @@
go-crypto is the cryptographic package adapted for Tendermint's uses
## Importing it
`import "github.com/tendermint/go-crypto"`
`import "github.com/tendermint/go-crypto"`

+ 4
- 4
_nano/keys.go View File

@ -9,7 +9,7 @@ import (
ledger "github.com/ethanfrey/ledger"
crypto "github.com/tendermint/go-crypto"
wire "github.com/tendermint/go-wire"
amino "github.com/tendermint/go-amino"
)
//nolint
@ -58,7 +58,7 @@ func signLedger(device *ledger.Ledger, msg []byte) (pub crypto.PubKey, sig crypt
// PrivKeyLedgerEd25519 implements PrivKey, calling the ledger nano
// we cache the PubKey from the first call to use it later
type PrivKeyLedgerEd25519 struct {
// PubKey should be private, but we want to encode it via go-wire
// PubKey should be private, but we want to encode it via go-amino
// so we can view the address later, even without having the ledger
// attached
CachedPubKey crypto.PubKey
@ -97,7 +97,7 @@ func (pk *PrivKeyLedgerEd25519) AssertIsPrivKeyInner() {}
// Bytes fulfils PrivKey Interface - but it stores the cached pubkey so we can verify
// the same key when we reconnect to a ledger
func (pk *PrivKeyLedgerEd25519) Bytes() []byte {
return wire.BinaryBytes(pk.Wrap())
return amino.BinaryBytes(pk.Wrap())
}
// Sign calls the ledger and stores the PubKey for future use
@ -250,7 +250,7 @@ func PubKeyLedgerEd25519FromBytes(key [32]byte) crypto.PubKey {
// Bytes fulfils pk Interface - no data, just type info
func (pk PubKeyLedgerEd25519) Bytes() []byte {
return wire.BinaryBytes(pk.Wrap())
return amino.BinaryBytes(pk.Wrap())
}
// VerifyBytes uses the normal Ed25519 algorithm but a sha512 hash beforehand


wire.go → amino.go View File


+ 8
- 8
encode_test.go View File

@ -1,7 +1,7 @@
package crypto
/*
XXX Needs to be refactored to not use go-wire/data
XXX Needs to be refactored to not use go-amino/data
import (
"fmt"
@ -10,15 +10,15 @@ import (
"github.com/stretchr/testify/assert"
"github.com/stretchr/testify/require"
wire "github.com/tendermint/go-wire"
data "github.com/tendermint/go-wire/data"
amino "github.com/tendermint/go-amino"
data "github.com/tendermint/go-amino/data"
)
type byter interface {
Bytes() []byte
}
// go to wire encoding and back
// go to amino encoding and back
func checkWire(t *testing.T, in byter, reader interface{}, typ byte, size int) {
// test to and from binary
bin, err := data.ToWire(in)
@ -55,15 +55,15 @@ func checkJSON(t *testing.T, in interface{}, reader interface{}, typ string) {
assert.True(t, strings.Contains(string(js), parts[1]))
}
// make sure go-wire json can still figure this out...
// make sure go-amino json can still figure this out...
func checkWireJSON(t *testing.T, in interface{}, reader interface{}, typ byte) {
// test to and from binary
var err error
js := wire.JSONBytes(in)
js := amino.JSONBytes(in)
btyp := fmt.Sprintf("[%d,", typ)
assert.True(t, strings.HasPrefix(string(js), btyp), string(js), btyp)
wire.ReadJSON(reader, js, &err)
amino.ReadJSON(reader, js, &err)
require.Nil(t, err, "%+v", err)
}
@ -144,7 +144,7 @@ type SigMessage struct {
}
func (s SigMessage) Bytes() []byte {
return wire.BinaryBytes(s)
return amino.BinaryBytes(s)
}
func TestEmbededWireEncodings(t *testing.T) {


+ 1
- 1
glide.lock View File

@ -59,7 +59,7 @@ imports:
subpackages:
- edwards25519
- extra25519
- name: github.com/tendermint/go-wire
- name: github.com/tendermint/go-amino
version: dec83f641903b22f039da3974607859715d0377e
- name: github.com/tendermint/tmlibs
version: 26f2ab65f82cfc6873c312e8030104c47c05f10e


+ 1
- 1
glide.yaml View File

@ -12,7 +12,7 @@ import:
- package: github.com/tendermint/ed25519
subpackages:
- extra25519
- package: github.com/tendermint/go-wire
- package: github.com/tendermint/go-amino
version: develop
- package: github.com/tendermint/tmlibs
version: develop


+ 4
- 4
keys/wire.go View File

@ -1,12 +1,12 @@
package keys
import (
"github.com/tendermint/go-crypto"
"github.com/tendermint/go-wire"
amino "github.com/tendermint/go-amino"
crypto "github.com/tendermint/go-crypto"
)
var cdc = wire.NewCodec()
var cdc = amino.NewCodec()
func init() {
crypto.RegisterWire(cdc)
crypto.RegisterAmino(cdc)
}

+ 10
- 9
signature_test.go View File

@ -6,7 +6,7 @@ import (
"github.com/stretchr/testify/assert"
"github.com/stretchr/testify/require"
"github.com/tendermint/ed25519"
"github.com/tendermint/go-wire"
amino "github.com/tendermint/go-amino"
)
func TestSignAndValidateEd25519(t *testing.T) {
@ -49,17 +49,17 @@ func TestSignatureEncodings(t *testing.T) {
cases := []struct {
privKey PrivKey
sigSize int
sigPrefix wire.PrefixBytes
sigPrefix amino.PrefixBytes
}{
{
privKey: GenPrivKeyEd25519(),
sigSize: ed25519.SignatureSize,
sigPrefix: [4]byte{0xe4, 0x51, 0x7b, 0xa3},
sigPrefix: [4]byte{0xc8, 0x5d, 0xf4, 0xba},
},
{
privKey: GenPrivKeySecp256k1(),
sigSize: 0, // unknown
sigPrefix: [4]byte{0x37, 0xb9, 0x21, 0x3e},
sigPrefix: [4]byte{0xc6, 0xa0, 0xa, 0x42},
},
}
@ -70,17 +70,18 @@ func TestSignatureEncodings(t *testing.T) {
msg := CRandBytes(128)
sig := tc.privKey.Sign(msg)
// store as wire
bin, err := cdc.MarshalBinary(sig)
// store as amino
bin, err := cdc.MarshalBinaryBare(sig)
require.Nil(t, err, "%+v", err)
if tc.sigSize != 0 {
assert.Equal(t, tc.sigSize+4, len(bin))
// Q: where is 1 byte coming from?
assert.Equal(t, tc.sigSize+amino.PrefixBytesLen+1, len(bin))
}
assert.Equal(t, tc.sigPrefix[:], bin[0:4])
assert.Equal(t, tc.sigPrefix[:], bin[0:amino.PrefixBytesLen])
// and back
sig2 := Signature(nil)
err = cdc.UnmarshalBinary(bin, &sig2)
err = cdc.UnmarshalBinaryBare(bin, &sig2)
require.Nil(t, err, "%+v", err)
assert.EqualValues(t, sig, sig2)
assert.True(t, pubKey.VerifyBytes(msg, sig2))


Loading…
Cancel
Save