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

Invalid simulation query on v0.40.1 #8425

Closed
4 tasks
nhannamsiu opened this issue Jan 25, 2021 · 1 comment · Fixed by #8926
Closed
4 tasks

Invalid simulation query on v0.40.1 #8425

nhannamsiu opened this issue Jan 25, 2021 · 1 comment · Fixed by #8926
Assignees
Labels
C: gRPC Issues and PRs related to the gRPC service and HTTP gateway. T:Bug
Milestone

Comments

@nhannamsiu
Copy link

Summary of Bug

client/tx.go calculateGas() uses query path /cosmos.tx.v1beta1.Service/Simulate to simulate the transaction. This query is routed to baseapp/abci.go Query(), which slipped through all cases and returned unknown query path.

Looks like it's work in progress though

// TODO This should use the generated tx service Client.
// https://github.com/cosmos/cosmos-sdk/issues/7726
bz, _, err := queryFunc("/cosmos.tx.v1beta1.Service/Simulate", txBytes)

A guide showing how to generate the tx service Client here would be very helpful.

Version

v0.40.1

Steps to Reproduce

  1. Follow the tutorial to setup basic app using cosmos v0.40.1.
  2. Make use of client/tx/tx.go broadcastTx(), which includes the calulateGas() and the error will show up.
  3. Sorry for vague description as our setup is complex, involves 3 repos to reproduce it and they are all private. We did our best to isolate the cause and hope others having same problem find this helpful.

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@clevinson
Copy link
Contributor

clevinson commented Jan 26, 2021

Assigning to @amaurymartiny to take a look at as you were the author of the referenced issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: gRPC Issues and PRs related to the gRPC service and HTTP gateway. T:Bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants