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

Export a reference API error if there is a chain id mismatch #51

Closed
DavidNix opened this issue Jun 1, 2023 · 2 comments
Closed

Export a reference API error if there is a chain id mismatch #51

DavidNix opened this issue Jun 1, 2023 · 2 comments

Comments

@DavidNix
Copy link
Contributor

DavidNix commented Jun 1, 2023

Right now we log a warning, but I think it should be an error that's exported (like we do with timeouts and bad status codes).

If there is a mismatch on chain id, we can't trust the data at all.

@jonathanpberger
Copy link
Contributor

@DavidNix what's the deal w/ this ticket?

@DavidNix
Copy link
Contributor Author

When we export the height metrics, if there's a chain id mismatch, we only log a warning. This should be an error we can report on if we want.

E.g. I want to use rest.cosmoshub.com for osmosis. The chain ids will be different so we are looking at heights for the wrong chain.

@akc2267 akc2267 closed this as completed Nov 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants