[Bug]: code using main branch cannot send transactions using cli command #18122

luchenqun · 2023-10-14T10:31:44Z

Is there an existing issue for this?

I have searched the existing issues

What happened?

First use the command simd testnet init-files --v 1 --output-dir ./nodes --chain-id simulation-app --keyring-backend test to initialize a test network
Then try to send a transfer transaction using the command simd tx bank send node0 cosmos1lh6pmahlkytz7lmrs25a42j6kxlmztyt204kyq 1stake --home ./nodes/node0/simd --keyring-backend test --chain-id simulation-app -y
The following error occurred:

invalid argument "node0" for "--0" flag: invalid account address or key name: decoding bech32 failed: invalid bech32 string length 5

Cosmos SDK Version

main

How to reproduce?

No response

The text was updated successfully, but these errors were encountered:

julienrbrt · 2023-10-14T12:58:26Z

Hi, is node0 is your keyring?
The error message means that node0 isn't a valid address or isn't a key name in the keyring.

Can you try to list the keys?

Can you verify that it uses the right home? (If not it is the same issue as here: #17822, having a fix incoming soon).

luchenqun · 2023-10-15T02:24:53Z

The account exists in the keyring-test directory

simd keys list --keyring-backend test --home ./nodes/node0/simd                                                                                           
- address: cosmos14gd8a9whwa95m6n8qcc7m3rtrx0ep0tjdgn7m8
  name: node0
  pubkey: '{"@type":"/cosmos.crypto.secp256k1.PubKey","key":"Amch1JQG486aee4I/Ey+cceRLAFq5pw8jKvK3QTVsUt3"}'
  type: local

luchenqun · 2023-10-15T02:48:59Z

In addition, for the same operation, there is no problem with tag v0.47.5, only the main branch has problems.

julienrbrt · 2023-10-15T05:07:33Z

Thank your for checking, I'll have a look 👍

tac0turtle · 2023-10-16T09:12:06Z

bank send works for me on main. I tried node0 with a the address you sent and it work fine for me as well. im unable to reproduce your error

luchenqun · 2023-10-16T13:20:46Z

Well, I just used the main branch (commit id is bb34c42) and it still doesn’t work. These are all the operations I did in the cosmos-sdk directory (I am on macos, the bold part is the command).

➜ cosmos-sdk git:(main) date
Mon Oct 16 21:09:43 CST 2023
➜ cosmos-sdk git:(main) make build
cd /Users/lcq/go/src/github.com/cosmos/cosmos-sdk/simapp && go build -mod=readonly -tags "netgo ledger" -ldflags '-X github.com/cosmos/cosmos-sdk/version.Name=sim -X github.com/cosmos/cosmos-sdk/version.AppName=simd -X github.com/cosmos/cosmos-sdk/version.Version=0.46.0-beta2-2745-gbb34c42f06 -X github.com/cosmos/cosmos-sdk/version.Commit=bb34c42f064d4c0b0e1db0d669c0e2f4139c879d -X "github.com/cosmos/cosmos-sdk/version.BuildTags=netgo,ledger" -w -s' -trimpath -o /Users/lcq/go/src/github.com/cosmos/cosmos-sdk/build/ ./...
➜ cosmos-sdk git:(main) cd build && ll simd
-rwxr-xr-x 1 lcq staff 92M Oct 16 21:09 simd
➜ build git:(simd) rm -rf nodes
➜ build git:(simd) ./simd testnet init-files --v 1 --output-dir ./nodes --chain-id simulation-app --keyring-backend test
Successfully initialized 1 node directories
➜ build git:(simd) ** nohup ./simd start --home ./nodes/node0/simd >./nodes/simd.log 2>&1 & **
[1] 18213
➜ build git:(simd) ./simd tx bank send node0 cosmos1gfg9ucc7rrzc207y9qfmf58erftzf8z8ww5lr7 1stake --home ./nodes/node0/simd --keyring-backend test --chain-id simulation-app -y
invalid argument "node0" for "--0" flag: invalid account address or key name: decoding bech32 failed: invalid bech32 string length 5
➜ build git:(simd) ./simd keys list --keyring-backend test --home ./nodes/node0/simd

address: cosmos1mta067yu6kkr93gzq4m9hettyuaqrlad3h9uzn
name: node0
pubkey: '{"@type":"/cosmos.crypto.secp256k1.PubKey","key":"A39Zn72TmoyLs7HVCqAUiUbySDty4CPF5YX2pZW2Scq3"}'
type: local

In addition, for the same operation, there is no problem with tag v0.47.5

➜ cosmos-sdk git:(main) git checkout v0.47.5
Note: switching to 'v0.47.5'.
.......
HEAD is now at 2e9e5d6 chore: prepare v0.47.5 (#17407)
➜ cosmos-sdk git:(2e9e5d6) make build
cd /Users/lcq/go/src/github.com/cosmos/cosmos-sdk/simapp && go build -mod=readonly -tags "netgo ledger" -ldflags '-X github.com/cosmos/cosmos-sdk/version.Name=sim -X github.com/cosmos/cosmos-sdk/version.AppName=simd -X github.com/cosmos/cosmos-sdk/version.Version=0.47.5 -X github.com/cosmos/cosmos-sdk/version.Commit=2e9e5d6eea24d6c11eddc9c002c66e89ae036187 -X "github.com/cosmos/cosmos-sdk/version.BuildTags=netgo,ledger" -X github.com/cometbft/cometbft/version.TMCoreSemVer=v0.37.2 -w -s' -trimpath -o /Users/lcq/go/src/github.com/cosmos/cosmos-sdk/build/ ./...
➜ cosmos-sdk git:(2e9e5d6) cd build
➜ build git:(simd) ./simd testnet init-files --v 1 --output-dir ./nodes --chain-id simulation-app --keyring-backend test
Successfully initialized 1 node directories
➜ build git:(simd) nohup ./simd start --home ./nodes/node0/simd >./nodes/simd.log 2>&1 &
[1] 19255
➜ build git:(simd) ./simd keys list --keyring-backend test --home ./nodes/node0/simd

address: cosmos17zy97lv9prfr2v934spuhp56zxh2nmjt57fxe3
name: node0
pubkey: '{"@type":"/cosmos.crypto.secp256k1.PubKey","key":"A0spAsYG3SL+3w05hJdJQUrvl/NNq2c8pM/8dhTsVCVq"}'
type: local

➜ build git:(simd) ./simd tx bank send node0 cosmos1gfg9ucc7rrzc207y9qfmf58erftzf8z8ww5lr7 1stake --home ./nodes/node0/simd --keyring-backend test --chain-id simulation-app -y
code: 0
codespace: ""
data: ""
events: []
gas_used: "0"
gas_wanted: "0"
height: "0"
info: ""
logs: []
raw_log: '[]'
timestamp: ""
tx: null
txhash: C6107394F3230361530941989C59935FF3FD98F87B4610AF191A17F516D8F2C6

julienrbrt · 2023-10-16T13:34:38Z

I, personally, can reproduce, looking into it as we speak.
This is because the keyring used to parse the input from the autocli flag handlers is created from the root.go and does not read any flag.

luchenqun · 2023-10-16T13:56:24Z

That's right. I think you probably haven't read the --home flag. When I use the default home, the main branch is no problem

➜ build git:(simd) ./simd tx bank send my_validator cosmos1lh6pmahlkytz7lmrs25a42j6kxlmztyt204kyq 1stake --keyring-backend test --chain-id simulation-app -y
code: 0
codespace: ""
data: ""
events: []
gas_used: "0"
gas_wanted: "0"
height: "0"
info: ""
logs: []
raw_log: ""
timestamp: ""
tx: null
txhash: CDF7187321851C413EB12029FA18FEC21717F43E713D6BDEC829F2D1A933ABCA
➜ build git:(simd) ./simd tx bank send node0 cosmos1gfg9ucc7rrzc207y9qfmf58erftzf8z8ww5lr7 1stake --home ./nodes/node0/simd --keyring-backend test --chain-id simulation-app -y
invalid argument "node0" for "--0" flag: invalid account address or key name: decoding bech32 failed: invalid bech32 string length 5

luchenqun · 2023-11-04T14:07:26Z

By the way, using --dry-run for simulation testing will not work either, fox exapmle: simd tx bank send node0 cosmos1gfg9ucc7rrzc207y9qfmf58erftzf8z8ww5lr7 1stake --keyring-backend test --chain-id simulation-app --gas auto --fees 129550stake --home ./nodes/node0/simd --dry-run. @julienrbrt
When do you plan to fix this bug?

julienrbrt · 2023-11-04T14:12:04Z

Im not in front of my laptop, but what message do you get after dry-run ?

This issue will be fixed next week.

luchenqun · 2023-11-04T14:54:04Z

The same problem occurred as above：invalid argument "node0" for "--0" flag: invalid account address or key name: decoding bech32 failed: invalid bech32 string length 5

luchenqun · 2023-11-04T14:57:49Z

I try to fix this problem but the cli has been basically refactored compared to the previous version v0.46.x, so I haven't found the problem after research for a long time.

julienrbrt · 2023-11-04T17:05:43Z

Yes, it is the same problem. Basically the keyring flags aren't working on main for AutoCLI commands. I know the reason, and I'll open a PR fixing it next week 👍

luchenqun · 2023-11-16T11:31:46Z

Looks like this issue hasn't been fixed yet?

julienrbrt · 2023-11-16T11:45:34Z

Yep yep. Other features on client/v2 got in first. This fix will come, nw.

luchenqun · 2023-11-19T03:53:01Z

So how do you send transactions use cli command?

julienrbrt · 2023-11-19T14:47:30Z

So how do you send transactions use cli command?

For reference: #18506 (comment)

theforager · 2023-12-27T22:15:51Z

Just +1'ing this issue. I ran into this error when going through the starter tutorial.

If a fix is hard, could we even just add a note on the tutorial so we don't lose folks?

julienrbrt · 2023-12-30T11:27:57Z

Just +1'ing this issue. I ran into this error when going through the starter tutorial.

If a fix is hard, could we even just add a note on the tutorial so we don't lose folks?

The tutorial shouldn't use main, so that shouldn't be an issue.

The fix isn't that hard, I am trying to make it with no wiring changes for chains, but that is harder.

I'd rather include it with the next few refactor of client/v2, but I am definitely still on it.

luchenqun added the T:Bug label Oct 14, 2023

julienrbrt self-assigned this Oct 14, 2023

julienrbrt mentioned this issue Oct 19, 2023

docs: update tutorials to work with SDK v0.50 ignite/cli#3705

Merged

julienrbrt added this to Cosmos-SDK Nov 1, 2023

github-project-automation bot moved this to 👀 To Do in Cosmos-SDK Nov 1, 2023

julienrbrt moved this from 👀 To Do to ✍ In Progress in Cosmos-SDK Nov 1, 2023

julienrbrt added the T:Sprint label Nov 1, 2023

julienrbrt mentioned this issue Nov 8, 2023

Hubl enhancements #18417

Open

4 tasks

tac0turtle removed the T:Sprint label Nov 16, 2023

luchenqun mentioned this issue Nov 19, 2023

imp(staking): detect the length of the ed25519 pubkey in CreateValidator to prevent panic #18506

Merged

czar0 mentioned this issue Nov 23, 2023

[Bug]: Error running simd tx bank send on main branch when following the IDA week 2 tutorial #18549

Closed

1 task

julienrbrt mentioned this issue Dec 22, 2023

[Bug]: --home flag is not respected #18868

Closed

1 task

This was referenced Jan 10, 2024

fix(simapp): home flag is not respected #18994

Merged

fix(client/v2): resolve keyring flags properly #19060

Merged

julienrbrt closed this as completed in #19060 Jan 18, 2024

github-project-automation bot moved this from ✍ In Progress to 🥳 Done in Cosmos-SDK Jan 18, 2024

julienrbrt mentioned this issue Feb 22, 2024

AutoCLI Enhancements #18310

Closed

5 tasks

tac0turtle removed this from Cosmos-SDK Feb 29, 2024

julienrbrt mentioned this issue Mar 4, 2024

[Bug]: tx bank send fails with invalid bech32 when using key alias #19645

Closed

1 task

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Bug]: code using main branch cannot send transactions using cli command #18122

[Bug]: code using main branch cannot send transactions using cli command #18122

luchenqun commented Oct 14, 2023

julienrbrt commented Oct 14, 2023 •

edited

Loading

luchenqun commented Oct 15, 2023

luchenqun commented Oct 15, 2023

julienrbrt commented Oct 15, 2023

tac0turtle commented Oct 16, 2023

luchenqun commented Oct 16, 2023 •

edited

Loading

julienrbrt commented Oct 16, 2023 •

edited

Loading

luchenqun commented Oct 16, 2023

luchenqun commented Nov 4, 2023 •

edited

Loading

julienrbrt commented Nov 4, 2023 •

edited

Loading

luchenqun commented Nov 4, 2023

luchenqun commented Nov 4, 2023

julienrbrt commented Nov 4, 2023

luchenqun commented Nov 16, 2023

julienrbrt commented Nov 16, 2023

luchenqun commented Nov 19, 2023

julienrbrt commented Nov 19, 2023

theforager commented Dec 27, 2023

julienrbrt commented Dec 30, 2023 •

edited

Loading

[Bug]: code using main branch cannot send transactions using cli command #18122

[Bug]: code using main branch cannot send transactions using cli command #18122

Comments

luchenqun commented Oct 14, 2023

Is there an existing issue for this?

What happened?

Cosmos SDK Version

How to reproduce?

julienrbrt commented Oct 14, 2023 • edited Loading

luchenqun commented Oct 15, 2023

luchenqun commented Oct 15, 2023

julienrbrt commented Oct 15, 2023

tac0turtle commented Oct 16, 2023

luchenqun commented Oct 16, 2023 • edited Loading

julienrbrt commented Oct 16, 2023 • edited Loading

luchenqun commented Oct 16, 2023

luchenqun commented Nov 4, 2023 • edited Loading

julienrbrt commented Nov 4, 2023 • edited Loading

luchenqun commented Nov 4, 2023

luchenqun commented Nov 4, 2023

julienrbrt commented Nov 4, 2023

luchenqun commented Nov 16, 2023

julienrbrt commented Nov 16, 2023

luchenqun commented Nov 19, 2023

julienrbrt commented Nov 19, 2023

theforager commented Dec 27, 2023

julienrbrt commented Dec 30, 2023 • edited Loading

julienrbrt commented Oct 14, 2023 •

edited

Loading

luchenqun commented Oct 16, 2023 •

edited

Loading

julienrbrt commented Oct 16, 2023 •

edited

Loading

luchenqun commented Nov 4, 2023 •

edited

Loading

julienrbrt commented Nov 4, 2023 •

edited

Loading

julienrbrt commented Dec 30, 2023 •

edited

Loading