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.

Sign up for GitHub

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

Jump to bottom

Query failed when specifying no height #2166

Closed
yito88 opened this issue Nov 13, 2023 · 0 comments · Fixed by #2167 or #2184
Closed

Query failed when specifying no height #2166

yito88 opened this issue Nov 13, 2023 · 0 comments · Fixed by #2167 or #2184
Labels
bug Something isn't working

Comments

@yito88
Copy link
Member

yito88 commented Nov 13, 2023 •
edited
Loading

v0.26.0

We replaced our RequestQuery with Tendermint's RequestQuery in #2141. The height of our RequestQuery was set by the latest height if it was Height 0 when converting from Tendermint's one. Now, the conversion has been removed, the RequestQuery has Height 0 for a query when specifying no height. When getting the existence proof, the function doesn't assume that the height is zero.

p.s. Other queries without specifying the height also failed after height storage_read_past_height_limit due to the same cause.
@yito88 yito88 added the bug Something isn't working label Nov 13, 2023
@yito88 yito88 mentioned this issue Nov 13, 2023
Merged
2 tasks
This was referenced Nov 14, 2023
Closed
Closed
@sug0 sug0 mentioned this issue Nov 16, 2023
Merged
2 tasks
@yito88 yito88 changed the title Proof query failed when specifying no height Query failed when specifying no height Nov 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
1 participant
@yito88