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

Rest endpoints appear to have gone away #2074

Closed
jtremback opened this issue Jan 24, 2023 · 4 comments
Closed

Rest endpoints appear to have gone away #2074

jtremback opened this issue Jan 24, 2023 · 4 comments
Assignees
Milestone

Comments

@jtremback
Copy link
Contributor

jtremback commented Jan 24, 2023

Summary of Bug

  • Some change in some rest endpoints, for example /blocks/latest, has made them not work
  • This would be a big issue for block explorers so we can’t release without addressing this
  • The odd thing is that the SDK version bump is a minor bump from 45.9 to 45.11, and was just supposed to add some RS related code to prepare for the release of Gaia v9
  • This needs to be summed up and presented to Marko or someone from the SDK team
@jtremback jtremback added this to the Gaia v8.0.0 milestone Jan 24, 2023
@julienrbrt
Copy link
Member

julienrbrt commented Jan 25, 2023

I have tried with simapp and these endpoints do work.

The only issue I can think about is cosmos/cosmos-sdk#13695, but it gives another error.
The above issue is due to an issue in the way a chain load its config. This had popped up in a point released because we shadowed a variable (which we've reverted in v0.45.12).
EDIT: I don't see an error with your config loading, so I don't think it's that.

@ryujimin220
Copy link

ryujimin220 commented Jan 25, 2023

We cannot provide services due to the same issue.

Tested Port

  • 1317 : not working
  • 26657 : good
  • 9090 grpc : good

Environment

  • OS : Ubuntu-2004-focal-64-minimal
  • Gaia : v8.0.0 RC3
  • Go : go1.19.5 linux/amd64

Both our archive and rpc nodes are facing this issue.

I do agree that this version should have been blocked from releasing.

@yaruwangway
Copy link
Contributor

tested with sdk 0.45.12 for v8:

the followings are there:

  • block?height=_
  • validators?height=&page=&per_page=_

the following are still missing:

  • blocks/latest
  • validatorsets/latest
  • node_info/synching

@jtremback
Copy link
Contributor Author

@julienrbrt sorry for assuming this was a Cosmos-SDK thing when it was indeed in our app

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

No branches or pull requests

5 participants