Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Submitting a PayForData tx from the cli fails the first time using the --gas-prices flag #379

Closed
Tracked by #411
evan-forbes opened this issue May 4, 2022 · 1 comment · Fixed by #419
Closed
Tracked by #411
Assignees
Labels
bug Something isn't working

Comments

@evan-forbes
Copy link
Member

evan-forbes commented May 4, 2022

The cli sub command fails when attempting to submit a pay for data transaction while using the --gas-prices flag. This can be recreated in the cli integration test and when using the latest version of master locally using the single-node.sh script located in contrib.

the input is

celestia-appd tx payment payForData "fffffffffffffffe" "68656c6c6f" --chain-id test --gas-prices "0.1uceles" --keyring-backend test --from=celes10824aaey7mc7ut4lgc8hpsqf6c37572ftzn9v5 --broadcast-mode block

the error returned is

raw_log: 'signature verification failed; please verify account number (0) and chain-id
  (test): unauthorized'
@evan-forbes
Copy link
Member Author

just to update on what I found today:

I no longer think that this is caused by adding specific flags, as the same command will create, sign, and broadcast a transaction just fine if ran again. It's only the first time that the transaction is ran do we get the error above.

The account number and chain-id that are being used as input to create a transaction are also what should be expected.

fwiw, this same error occurs with the #399 branch that updates the cosmos-sdk to v0.46.0

@evan-forbes evan-forbes changed the title Submitting a PayForData tx from the cli fails when using --gas-prices flag Submitting a PayForData tx from the cli fails the first time using the --gas-prices flag May 16, 2022
@adlerjohn adlerjohn linked a pull request May 19, 2022 that will close this issue
Repository owner moved this from TODO to Done in Celestia Node May 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
No open projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant