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

Legacy amino tx queries by height fail. #7721

Closed
4 tasks
zmanian opened this issue Oct 29, 2020 · 2 comments · Fixed by #7730
Closed
4 tasks

Legacy amino tx queries by height fail. #7721

zmanian opened this issue Oct 29, 2020 · 2 comments · Fixed by #7730
Assignees
Labels

Comments

@zmanian
Copy link
Member

zmanian commented Oct 29, 2020

$ curl -v '34.123.30.100:1317/txs?limit=100&page=1&tx.height=200403'
*   Trying 34.123.30.100...
* TCP_NODELAY set
* Connected to 34.123.30.100 (34.123.30.100) port 1317 (#0)
> GET /txs?limit=100&page=1&tx.height=200403 HTTP/1.1
> Host: 34.123.30.100:1317
> User-Agent: curl/7.64.1
> Accept: */*
>
< HTTP/1.1 500 Internal Server Error
< Content-Type: application/json
< X-Server-Time: 1603928732
< Date: Wed, 28 Oct 2020 23:45:32 GMT
< Content-Length: 52
<
* Connection #0 to host 34.123.30.100 left intact
{"error":"Unregistered interface tx.isModeInfo_Sum"}* Closing connection 0

This is on cosmos-sdk v0.40.rc-0 ie. stargate 4. It's possible that upstream fixes on Master may have helped.

We integration partners that use this options.

For Admin Use

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

clevinson commented Oct 29, 2020

Thanks for reporting. I've confirmed that this is still an issue on master, testing against a fresh local network with make localnet-start, and performing these queries after submitting a few dummy transactions.

This is not limited to just querying by height, but I also verified that the issue exists when querying for tx by sender on the same endpoint:

 curl -v 'localhost:1317/txs?limit=100&page=1&message.sender=cosmos1scn6psrral8q22v2acn23pad6l72c67l34jymw'

This returns with the exact same error. "Unregistered interface tx.isModeInfo_Sum"

@clevinson clevinson added this to the v0.40 [Stargate] milestone Oct 29, 2020
@aaronc
Copy link
Member

aaronc commented Oct 29, 2020

That's an amino error. Looks like we didn't even convert that endpoint... not sure how it got missed it's in the same file as the other one

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants