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

API Node Returns incomplete Block Data #4057

Closed
freedombird9 opened this issue Nov 26, 2024 · 5 comments · Fixed by #4059
Closed

API Node Returns incomplete Block Data #4057

freedombird9 opened this issue Nov 26, 2024 · 5 comments · Fixed by #4059

Comments

@freedombird9
Copy link

After upgarding to celestia-app v3.0.1, query query /cosmos/tx/v1beta1/txs?events=tx.height=2876500&page=1&limit=100 only returns the first transaction: 84D8A4D3A6461BA7D57B0A3214C16C4F268ABA2F902C85124A928A7B5CE812D1

The other 7 transactions are not returned. We're following https://github.com/celestiaorg/celestia-app/releases/tag/v3.0.1 to upgrade our node. The public API node does return the correct data.

@mindstyle85
Copy link

hmm. i think issues still arent fixed @evan-forbes , ive also seen another report on discord

@rootulp
Copy link
Collaborator

rootulp commented Nov 26, 2024

Thanks for opening the issue @freedombird9 and reporting @mindstyle85 .

This issue looks like celestiaorg/celestia-core#1526 which was fixed by celestiaorg/celestia-core#1533 but it hasn't been released. I just cut celestia-core https://github.com/celestiaorg/celestia-core/releases/tag/v1.44.0-tm-v0.34.35 and bumped to it in #4059.

@freedombird9 can you confirm this PR fixes your issue? If you tell me which platform and architecture you're on, I can generate a binary for you. Alternatively, you can checkout the code on #4059, generate the binary (via make build or make install) and then re-run your node and the query.

@mindstyle85
Copy link

oh i see, i thought this was fixed in 3.0.1 hence why i pinged so fast

@rootulp
Copy link
Collaborator

rootulp commented Nov 26, 2024

We'll try to release it in v3.0.2 ASAP.

@rootulp
Copy link
Collaborator

rootulp commented Nov 28, 2024

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

Successfully merging a pull request may close this issue.

3 participants