accounts/abi/bind: allow specifying signer on transactOpts (ethereum#…

…21356)

This commit enables users to specify which signer they want to use while creating their transactOpts.
Previously all contract interactions used the homestead signer. Now a user can specify whether they
want to sign with homestead or EIP155 and specify the chainID which adds another layer of security.

Closes ethereum#16484

accounts/abi/bind/auth.go

@@ -21,18 +21,29 @@ import (
  "errors"
  "io"
  "io/ioutil"
+ "math/big"
 
  "github.com/ethereum/go-ethereum/accounts"
  "github.com/ethereum/go-ethereum/accounts/external"
  "github.com/ethereum/go-ethereum/accounts/keystore"
  "github.com/ethereum/go-ethereum/common"
  "github.com/ethereum/go-ethereum/core/types"
  "github.com/ethereum/go-ethereum/crypto"
+ "github.com/ethereum/go-ethereum/log"
 )
 
+// ErrNoChainID is returned whenever the user failed to specify a chain id.
+var ErrNoChainID = errors.New("no chain id specified")
+
+// ErrNotAuthorized is returned when an account is not properly unlocked.
+var ErrNotAuthorized = errors.New("not authorized to sign this account")
+
 // NewTransactor is a utility method to easily create a transaction signer from
 // an encrypted json key stream and the associated passphrase.
+//
+// Deprecated: Use NewTransactorWithChainID instead.
 func NewTransactor(keyin io.Reader, passphrase string) (*TransactOpts, error) {
+ log.Warn("WARNING: NewTransactor has been deprecated in favour of NewTransactorWithChainID")
  json, err := ioutil.ReadAll(keyin)
  if err != nil {
  return nil, err
@@ -45,13 +56,17 @@ func NewTransactor(keyin io.Reader, passphrase string) (*TransactOpts, error) {
 }
 
 // NewKeyStoreTransactor is a utility method to easily create a transaction signer from
-// a decrypted key from a keystore.
+// an decrypted key from a keystore.
+//
+// Deprecated: Use NewKeyStoreTransactorWithChainID instead.
 func NewKeyStoreTransactor(keystore *keystore.KeyStore, account accounts.Account) (*TransactOpts, error) {
+ log.Warn("WARNING: NewKeyStoreTransactor has been deprecated in favour of NewTransactorWithChainID")
+ signer := types.HomesteadSigner{}
  return &TransactOpts{
  From: account.Address,
- Signer: func(signer types.Signer, address common.Address, tx *types.Transaction) (*types.Transaction, error) {
+ Signer: func(address common.Address, tx *types.Transaction) (*types.Transaction, error) {
  if address != account.Address {
- return nil, errors.New("not authorized to sign this account")
+ return nil, ErrNotAuthorized
  }
  signature, err := keystore.SignHash(account, signer.Hash(tx).Bytes())
  if err != nil {
@@ -64,13 +79,17 @@ func NewKeyStoreTransactor(keystore *keystore.KeyStore, account accounts.Account
 
 // NewKeyedTransactor is a utility method to easily create a transaction signer
 // from a single private key.
+//
+// Deprecated: Use NewKeyedTransactorWithChainID instead.
 func NewKeyedTransactor(key *ecdsa.PrivateKey) *TransactOpts {
+ log.Warn("WARNING: NewKeyedTransactor has been deprecated in favour of NewKeyedTransactorWithChainID")
  keyAddr := crypto.PubkeyToAddress(key.PublicKey)
+ signer := types.HomesteadSigner{}
  return &TransactOpts{
  From: keyAddr,
- Signer: func(signer types.Signer, address common.Address, tx *types.Transaction) (*types.Transaction, error) {
+ Signer: func(address common.Address, tx *types.Transaction) (*types.Transaction, error) {
  if address != keyAddr {
- return nil, errors.New("not authorized to sign this account")
+ return nil, ErrNotAuthorized
  }
  signature, err := crypto.Sign(signer.Hash(tx).Bytes(), key)
  if err != nil {
@@ -81,14 +100,73 @@ func NewKeyedTransactor(key *ecdsa.PrivateKey) *TransactOpts {
  }
 }
 
+// NewTransactorWithChainID is a utility method to easily create a transaction signer from
+// an encrypted json key stream and the associated passphrase.
+func NewTransactorWithChainID(keyin io.Reader, passphrase string, chainID *big.Int) (*TransactOpts, error) {
+ json, err := ioutil.ReadAll(keyin)
+ if err != nil {
+ return nil, err
+ }
+ key, err := keystore.DecryptKey(json, passphrase)
+ if err != nil {
+ return nil, err
+ }
+ return NewKeyedTransactorWithChainID(key.PrivateKey, chainID)
+}
+
+// NewKeyStoreTransactorWithChainID is a utility method to easily create a transaction signer from
+// an decrypted key from a keystore.
+func NewKeyStoreTransactorWithChainID(keystore *keystore.KeyStore, account accounts.Account, chainID *big.Int) (*TransactOpts, error) {
+ if chainID == nil {
+ return nil, ErrNoChainID
+ }
+ signer := types.NewEIP155Signer(chainID)
+ return &TransactOpts{
+ From: account.Address,
+ Signer: func(address common.Address, tx *types.Transaction) (*types.Transaction, error) {
+ if address != account.Address {
+ return nil, ErrNotAuthorized
+ }
+ signature, err := keystore.SignHash(account, signer.Hash(tx).Bytes())
+ if err != nil {
+ return nil, err
+ }
+ return tx.WithSignature(signer, signature)
+ },
+ }, nil
+}
+
+// NewKeyedTransactorWithChainID is a utility method to easily create a transaction signer
+// from a single private key.
+func NewKeyedTransactorWithChainID(key *ecdsa.PrivateKey, chainID *big.Int) (*TransactOpts, error) {
+ keyAddr := crypto.PubkeyToAddress(key.PublicKey)
+ if chainID == nil {
+ return nil, ErrNoChainID
+ }
+ signer := types.NewEIP155Signer(chainID)
+ return &TransactOpts{
+ From: keyAddr,
+ Signer: func(address common.Address, tx *types.Transaction) (*types.Transaction, error) {
+ if address != keyAddr {
+ return nil, ErrNotAuthorized
+ }
+ signature, err := crypto.Sign(signer.Hash(tx).Bytes(), key)
+ if err != nil {
+ return nil, err
+ }
+ return tx.WithSignature(signer, signature)
+ },
+ }, nil
+}
+
 // NewClefTransactor is a utility method to easily create a transaction signer
 // with a clef backend.
 func NewClefTransactor(clef *external.ExternalSigner, account accounts.Account) *TransactOpts {
  return &TransactOpts{
  From: account.Address,
- Signer: func(signer types.Signer, address common.Address, transaction *types.Transaction) (*types.Transaction, error) {
+ Signer: func(address common.Address, transaction *types.Transaction) (*types.Transaction, error) {
  if address != account.Address {
- return nil, errors.New("not authorized to sign this account")
+ return nil, ErrNotAuthorized
  }
  return clef.SignTx(account, transaction, nil) // Clef enforces its own chain id
  },

accounts/abi/bind/backends/simulated.go

@@ -74,6 +74,7 @@ type SimulatedBackend struct {
 
 // NewSimulatedBackendWithDatabase creates a new binding backend based on the given database
 // and uses a simulated blockchain for testing purposes.
+// A simulated backend always uses chainID 1337.
 func NewSimulatedBackendWithDatabase(database ethdb.Database, alloc core.GenesisAlloc, gasLimit uint64) *SimulatedBackend {
  genesis := core.Genesis{Config: params.AllEthashProtocolChanges, GasLimit: gasLimit, Alloc: alloc}
  genesis.MustCommit(database)
@@ -91,6 +92,7 @@ func NewSimulatedBackendWithDatabase(database ethdb.Database, alloc core.Genesis
 
 // NewSimulatedBackend creates a new binding backend using a simulated blockchain
 // for testing purposes.
+// A simulated backend always uses chainID 1337.
 func NewSimulatedBackend(alloc core.GenesisAlloc, gasLimit uint64) *SimulatedBackend {
  return NewSimulatedBackendWithDatabase(rawdb.NewMemoryDatabase(), alloc, gasLimit)
 }

accounts/abi/bind/backends/simulated_test.go

@@ -39,7 +39,7 @@ import (
 func TestSimulatedBackend(t *testing.T) {
  var gasLimit uint64 = 8000029
  key, _ := crypto.GenerateKey() // nolint: gosec
- auth := bind.NewKeyedTransactor(key)
+ auth, _ := bind.NewKeyedTransactorWithChainID(key, big.NewInt(1337))
  genAlloc := make(core.GenesisAlloc)
  genAlloc[auth.From] = core.GenesisAccount{Balance: big.NewInt(9223372036854775807)}
 
@@ -411,7 +411,7 @@ func TestSimulatedBackend_EstimateGas(t *testing.T) {
 
  key, _ := crypto.GenerateKey()
  addr := crypto.PubkeyToAddress(key.PublicKey)
- opts := bind.NewKeyedTransactor(key)
+ opts, _ := bind.NewKeyedTransactorWithChainID(key, big.NewInt(1337))
 
  sim := NewSimulatedBackend(core.GenesisAlloc{addr: {Balance: big.NewInt(params.Ether)}}, 10000000)
  defer sim.Close()
@@ -888,7 +888,7 @@ func TestSimulatedBackend_PendingCodeAt(t *testing.T) {
  if err != nil {
  t.Errorf("could not get code at test addr: %v", err)
  }
- auth := bind.NewKeyedTransactor(testKey)
+ auth, _ := bind.NewKeyedTransactorWithChainID(testKey, big.NewInt(1337))
  contractAddr, tx, contract, err := bind.DeployContract(auth, parsed, common.FromHex(abiBin), sim)
  if err != nil {
  t.Errorf("could not deploy contract: %v tx: %v contract: %v", err, tx, contract)
@@ -924,7 +924,7 @@ func TestSimulatedBackend_CodeAt(t *testing.T) {
  if err != nil {
  t.Errorf("could not get code at test addr: %v", err)
  }
- auth := bind.NewKeyedTransactor(testKey)
+ auth, _ := bind.NewKeyedTransactorWithChainID(testKey, big.NewInt(1337))
  contractAddr, tx, contract, err := bind.DeployContract(auth, parsed, common.FromHex(abiBin), sim)
  if err != nil {
  t.Errorf("could not deploy contract: %v tx: %v contract: %v", err, tx, contract)
@@ -956,7 +956,7 @@ func TestSimulatedBackend_PendingAndCallContract(t *testing.T) {
  if err != nil {
  t.Errorf("could not get code at test addr: %v", err)
  }
- contractAuth := bind.NewKeyedTransactor(testKey)
+ contractAuth, _ := bind.NewKeyedTransactorWithChainID(testKey, big.NewInt(1337))
  addr, _, _, err := bind.DeployContract(contractAuth, parsed, common.FromHex(abiBin), sim)
  if err != nil {
  t.Errorf("could not deploy contract: %v", err)
@@ -1043,7 +1043,7 @@ func TestSimulatedBackend_CallContractRevert(t *testing.T) {
  if err != nil {
  t.Errorf("could not get code at test addr: %v", err)
  }
- contractAuth := bind.NewKeyedTransactor(testKey)
+ contractAuth, _ := bind.NewKeyedTransactorWithChainID(testKey, big.NewInt(1337))
  addr, _, _, err := bind.DeployContract(contractAuth, parsed, common.FromHex(reverterBin), sim)
  if err != nil {
  t.Errorf("could not deploy contract: %v", err)

accounts/abi/bind/base.go

@@ -32,7 +32,7 @@ import (
 
 // SignerFn is a signer function callback when a contract requires a method to
 // sign the transaction before submission.
-type SignerFn func(types.Signer, common.Address, *types.Transaction) (*types.Transaction, error)
+type SignerFn func(common.Address, *types.Transaction) (*types.Transaction, error)
 
 // CallOpts is the collection of options to fine tune a contract call request.
 type CallOpts struct {
@@ -256,7 +256,7 @@ func (c *BoundContract) transact(opts *TransactOpts, contract *common.Address, i
  if opts.Signer == nil {
  return nil, errors.New("no signer to authorize the transaction with")
  }
- signedTx, err := opts.Signer(types.HomesteadSigner{}, opts.From, rawTx)
+ signedTx, err := opts.Signer(opts.From, rawTx)
  if err != nil {
  return nil, err
  }

accounts/abi/bind/bind_test.go

@@ -296,7 +296,7 @@ var bindTests = []struct {
  `
  // Generate a new random account and a funded simulator
  key, _ := crypto.GenerateKey()
- auth := bind.NewKeyedTransactor(key)
+ auth, _ := bind.NewKeyedTransactorWithChainID(key, big.NewInt(1337))
 
  sim := backends.NewSimulatedBackend(core.GenesisAlloc{auth.From: {Balance: big.NewInt(10000000000)}}, 10000000)
  defer sim.Close()
@@ -351,7 +351,7 @@ var bindTests = []struct {
  `
  // Generate a new random account and a funded simulator
  key, _ := crypto.GenerateKey()
- auth := bind.NewKeyedTransactor(key)
+ auth, _ := bind.NewKeyedTransactorWithChainID(key, big.NewInt(1337))
 
  sim := backends.NewSimulatedBackend(core.GenesisAlloc{auth.From: {Balance: big.NewInt(10000000000)}}, 10000000)
  defer sim.Close()
@@ -397,7 +397,7 @@ var bindTests = []struct {
  `
  // Generate a new random account and a funded simulator
  key, _ := crypto.GenerateKey()
- auth := bind.NewKeyedTransactor(key)
+ auth, _ := bind.NewKeyedTransactorWithChainID(key, big.NewInt(1337))
 
  sim := backends.NewSimulatedBackend(core.GenesisAlloc{auth.From: {Balance: big.NewInt(10000000000)}}, 10000000)
  defer sim.Close()
@@ -455,7 +455,7 @@ var bindTests = []struct {
  `
  // Generate a new random account and a funded simulator
  key, _ := crypto.GenerateKey()
- auth := bind.NewKeyedTransactor(key)
+ auth, _ := bind.NewKeyedTransactorWithChainID(key, big.NewInt(1337))
 
  sim := backends.NewSimulatedBackend(core.GenesisAlloc{auth.From: {Balance: big.NewInt(10000000000)}}, 10000000)
  defer sim.Close()
@@ -503,7 +503,7 @@ var bindTests = []struct {
  `
  // Generate a new random account and a funded simulator
  key, _ := crypto.GenerateKey()
- auth := bind.NewKeyedTransactor(key)
+ auth, _ := bind.NewKeyedTransactorWithChainID(key, big.NewInt(1337))
 
  sim := backends.NewSimulatedBackend(core.GenesisAlloc{auth.From: {Balance: big.NewInt(10000000000)}}, 10000000)
  defer sim.Close()
@@ -598,7 +598,7 @@ var bindTests = []struct {
  `
  // Generate a new random account and a funded simulator
  key, _ := crypto.GenerateKey()
- auth := bind.NewKeyedTransactor(key)
+ auth, _ := bind.NewKeyedTransactorWithChainID(key, big.NewInt(1337))
 
  sim := backends.NewSimulatedBackend(core.GenesisAlloc{auth.From: {Balance: big.NewInt(10000000000)}}, 10000000)
  defer sim.Close()
@@ -648,7 +648,7 @@ var bindTests = []struct {
  `
  // Generate a new random account and a funded simulator
  key, _ := crypto.GenerateKey()
- auth := bind.NewKeyedTransactor(key)
+ auth, _ := bind.NewKeyedTransactorWithChainID(key, big.NewInt(1337))
 
  sim := backends.NewSimulatedBackend(core.GenesisAlloc{auth.From: {Balance: big.NewInt(10000000000)}}, 10000000)
  defer sim.Close()
@@ -723,7 +723,7 @@ var bindTests = []struct {
  `
  // Generate a new random account and a funded simulator
  key, _ := crypto.GenerateKey()
- auth := bind.NewKeyedTransactor(key)
+ auth, _ := bind.NewKeyedTransactorWithChainID(key, big.NewInt(1337))
 
  sim := backends.NewSimulatedBackend(core.GenesisAlloc{auth.From: {Balance: big.NewInt(10000000000)}}, 10000000)
  defer sim.Close()
@@ -817,7 +817,7 @@ var bindTests = []struct {
  `
  // Generate a new random account and a funded simulator
  key, _ := crypto.GenerateKey()
- auth := bind.NewKeyedTransactor(key)
+ auth, _ := bind.NewKeyedTransactorWithChainID(key, big.NewInt(1337))
 
  sim := backends.NewSimulatedBackend(core.GenesisAlloc{auth.From: {Balance: big.NewInt(10000000000)}}, 10000000)
  defer sim.Close()
@@ -1007,7 +1007,7 @@ var bindTests = []struct {
  `
  // Generate a new random account and a funded simulator
  key, _ := crypto.GenerateKey()
- auth := bind.NewKeyedTransactor(key)
+ auth, _ := bind.NewKeyedTransactorWithChainID(key, big.NewInt(1337))
 
  sim := backends.NewSimulatedBackend(core.GenesisAlloc{auth.From: {Balance: big.NewInt(10000000000)}}, 10000000)
  defer sim.Close()
@@ -1142,7 +1142,7 @@ var bindTests = []struct {
 
  `
  key, _ := crypto.GenerateKey()
- auth := bind.NewKeyedTransactor(key)
+ auth, _ := bind.NewKeyedTransactorWithChainID(key, big.NewInt(1337))
 
  sim := backends.NewSimulatedBackend(core.GenesisAlloc{auth.From: {Balance: big.NewInt(10000000000)}}, 10000000)
  defer sim.Close()
@@ -1284,7 +1284,7 @@ var bindTests = []struct {
  `
  // Generate a new random account and a funded simulator
  key, _ := crypto.GenerateKey()
- auth := bind.NewKeyedTransactor(key)
+ auth, _ := bind.NewKeyedTransactorWithChainID(key, big.NewInt(1337))
 
  sim := backends.NewSimulatedBackend(core.GenesisAlloc{auth.From: {Balance: big.NewInt(10000000000)}}, 10000000)
  defer sim.Close()
@@ -1350,7 +1350,7 @@ var bindTests = []struct {
  `
  // Initialize test accounts
  key, _ := crypto.GenerateKey()
- auth := bind.NewKeyedTransactor(key)
+ auth, _ := bind.NewKeyedTransactorWithChainID(key, big.NewInt(1337))
  sim := backends.NewSimulatedBackend(core.GenesisAlloc{auth.From: {Balance: big.NewInt(10000000000)}}, 10000000)
  defer sim.Close()
 
@@ -1444,7 +1444,7 @@ var bindTests = []struct {
  sim := backends.NewSimulatedBackend(core.GenesisAlloc{addr: {Balance: big.NewInt(1000000000)}}, 10000000)
  defer sim.Close()
 
- transactOpts := bind.NewKeyedTransactor(key)
+ transactOpts, _ := bind.NewKeyedTransactorWithChainID(key, big.NewInt(1337))
  _, _, _, err := DeployIdentifierCollision(transactOpts, sim)
  if err != nil {
  t.Fatalf("failed to deploy contract: %v", err)
@@ -1506,7 +1506,7 @@ var bindTests = []struct {
  sim := backends.NewSimulatedBackend(core.GenesisAlloc{addr: {Balance: big.NewInt(1000000000)}}, 10000000)
  defer sim.Close()
 
- transactOpts := bind.NewKeyedTransactor(key)
+ transactOpts, _ := bind.NewKeyedTransactorWithChainID(key, big.NewInt(1337))
  _, _, c1, err := DeployContractOne(transactOpts, sim)
  if err != nil {
  t.Fatal("Failed to deploy contract")
@@ -1563,7 +1563,7 @@ var bindTests = []struct {
  `
  // Generate a new random account and a funded simulator
  key, _ := crypto.GenerateKey()
- auth := bind.NewKeyedTransactor(key)
+ auth, _ := bind.NewKeyedTransactorWithChainID(key, big.NewInt(1337))
 
  sim := backends.NewSimulatedBackend(core.GenesisAlloc{auth.From: {Balance: big.NewInt(10000000000)}}, 10000000)
  defer sim.Close()
@@ -1632,7 +1632,7 @@ var bindTests = []struct {
  sim := backends.NewSimulatedBackend(core.GenesisAlloc{addr: {Balance: big.NewInt(1000000000)}}, 1000000)
  defer sim.Close()
  
- opts := bind.NewKeyedTransactor(key)
+ opts, _ := bind.NewKeyedTransactorWithChainID(key, big.NewInt(1337))
  _, _, c, err := DeployNewFallbacks(opts, sim)
  if err != nil {
  t.Fatalf("Failed to deploy contract: %v", err)

contracts/checkpointoracle/oracle_test.go

@@ -178,7 +178,7 @@ func TestCheckpointRegister(t *testing.T) {
  contractBackend := backends.NewSimulatedBackend(core.GenesisAlloc{accounts[0].addr: {Balance: big.NewInt(1000000000)}, accounts[1].addr: {Balance: big.NewInt(1000000000)}, accounts[2].addr: {Balance: big.NewInt(1000000000)}}, 10000000)
  defer contractBackend.Close()
 
- transactOpts := bind.NewKeyedTransactor(accounts[0].key)
+ transactOpts, _ := bind.NewKeyedTransactorWithChainID(accounts[0].key, big.NewInt(1337))
 
  // 3 trusted signers, threshold 2
  contractAddr, _, c, err := contract.DeployCheckpointOracle(transactOpts, contractBackend, []common.Address{accounts[0].addr, accounts[1].addr, accounts[2].addr}, sectionSize, processConfirms, big.NewInt(2))