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

Update App AnteHandler #83

Merged
merged 1 commit into from
Aug 19, 2019
Merged

Conversation

austinabell
Copy link

  • Changes from default antehandler to AnteHandler that was built for ethereum transactions
  • Adds a temporary measure of allowing tendermint Key to sign genesis transactions to be able to fund a validator to start the node to not have to build out Implement Genesis utility modules #81 as it is out of the scope of the current work
  • Allows the gas consumption logic to be included and usable when implementing the other functionality of an Ethereum transaction within the CosmosSDK handler framework for a tx

This includes the functionality of the Application previously, allowing the previous flow for starting the app, for example as so:

make install 
rm -rf ~/.emint*
emintd init moniker --chain-id testchain
emintcli config chain-id testchain
emintcli config output json
emintcli config indent true
emintcli config trust-node true
emintcli keys add austin
<password>
<password>
emintd add-genesis-account $(emintcli keys show austin -a) 1000photon,100000000stake
emintd gentx --name austin
<password>
emintd collect-gentxs
emintd validate-genesis
emintd start

Implements #71, and I will create an issue to remove the workaround for the genesis tx and find a solution that is compatible for anyone using the evm module.

Copy link

@ansermino ansermino left a comment

Choose a reason for hiding this comment

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

👍

I do get this error when I hit ctl-c, I suspect its negligible though:

E[2019-08-19|10:52:00.657] Not stopping BlockPool -- have not been started yet module=blockchain impl=BlockPool

@austinabell austinabell merged commit f0e2c01 into development Aug 19, 2019
@austinabell austinabell deleted the austin/updateantehandler branch August 19, 2019 15:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants