TruStory / truchain

⛓ A crypto-incentivized debate community

http://www.trustory.io

Other

33 stars 14 forks source link

Re-enabled ante-handler #780

Closed shanev closed 5 years ago

shanev commented 5 years ago

Fixes #364. Fixes #776.

Let's try this out on devnet.

codecov[bot] commented 5 years ago

Codecov Report

Merging #780 into develop will not change coverage. The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #780   +/-   ##
========================================
  Coverage    58.64%   58.64%           
========================================
  Files           73       73           
  Lines         4239     4239           
========================================
  Hits          2486     2486           
  Misses        1536     1536           
  Partials       217      217

truted commented 5 years ago

Fixes #364. Fixes #776.

Let's try this out on devnet.

Can we enable this using an ENV or cli command (i.e. truchaind --ante-handler --home ~/.truchaind. My worry is that once we merge this we won't be able to upgrade beta without enabling this in production.

shanev commented 5 years ago

@truted Going to hold off on this for a bit.

shanev commented 5 years ago

When registering a new account:

Response:
  TxHash: 110A0ECC755BE96BB20D09AE232A1790C169CCEF0D9FF35DD1B798AF796B4EE2
  Code: 8
  Raw Log: {"codespace":"sdk","code":8,"message":"PubKey does not match Signer address cosmos1w3e82cmgv95kuctrvdex2emfwd68yctjpzp3mr"}
  GasUsed: 3552

shanev commented 5 years ago

Closed in favor of https://github.com/TruStory/truchain/pull/839