cannot protobuf JSON encode unsupported type: *tx.builder

[icafa]

Latest master: 22e874a

SDK's cli based tx sending is broken.

How to reproduce.

start node

make build-simd
rm -rf $HOME/.simapp/
./build/simd init --chain-id=testing testing
./build/simd keys add validator --keyring-backend=test
./build/simd add-genesis-account $(./build/simd keys show validator -a --keyring-backend=test) 1000000000stake,1000000000validatortoken --keyring-backend=test
./build/simd gentx validator --keyring-backend=test --chain-id=testing
./build/simd collect-gentxs
./build/simd start

Try bank send
Test bank module.

./build/simd tx bank send validator cosmos1f339fylyn4czjz06l2s0laxa7377yr72qp880u 1000stake --keyring-backend=test --chain-id=testing
Error: cannot protobuf JSON encode unsupported type: *tx.builder
Usage:
  simd tx bank send [from_key_or_address] [to_address] [amount] [flags]

Flags:
  -a, --account-number uint      The account number of the signing account (offline mode only)
  -b, --broadcast-mode string    Transaction broadcasting mode (sync|async|block) (default "sync")
      --dry-run                  ignore the --gas flag and perform a simulation of a transaction, but don't broadcast it
      --fees string              Fees to pay along with transaction; eg: 10uatom
      --from string              Name or address of private key with which to sign
      --gas string               gas limit to set per-transaction; set to "auto" to calculate sufficient gas automatically (default 200000)
      --gas-adjustment float     adjustment factor to be multiplied against the estimate returned by the tx simulation; if the gas limit is set manually this flag is ignored  (default 1)
      --gas-prices string        Gas prices in decimal format to determine the transaction fee (e.g. 0.1uatom)
      --generate-only            Build an unsigned transaction and write it to STDOUT (when enabled, the local Keybase is not accessible)
  -h, --help                     help for send
      --keyring-backend string   Select keyring's backend (os|file|kwallet|pass|test) (default "os")
      --ledger                   Use a connected Ledger device
      --memo string              Memo to send along with transaction
      --node string              <host>:<port> to tendermint rpc interface for this chain (default "tcp://localhost:26657")
      --offline                  Offline mode (does not allow any online functionality
  -s, --sequence uint            The sequence number of the signing account (offline mode only)
      --sign-mode string         Choose sign mode (direct|amino-json), this is an advanced feature
      --timeout-height uint      Set a block timeout height to prevent the tx from being committed past a certain height
  -y, --yes                      Skip tx broadcasting prompt confirmation

Global Flags:
      --chain-id string   The network chain ID
      --home string       directory for config and data (default "/Users/admin/.simapp")
      --trace             print out full stack trace on errors

It will be good to have test to check this issue on next commit.

[aaronc]

Wasn't this fixed already @sahith-narahari ?

[icafa]

Let me know when it's fixed, @aaronc , it's blocker for us. We use latest master.

[sahith-narahari]

@junkai121 it should be fixed in master now, please check and confirm

[icafa]

I confirm it work @sahith-narahari Thanks for fast fix