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

Better error message when requesting a block that isn't there. #369

Closed
winder opened this issue Mar 1, 2021 · 5 comments · Fixed by #719
Closed

Better error message when requesting a block that isn't there. #369

winder opened this issue Mar 1, 2021 · 5 comments · Fixed by #719

Comments

@winder
Copy link
Contributor

winder commented Mar 1, 2021

Summary

When a block is missing from the indexer (i.e. requesting a round we haven't reached yet), the error message says "SQL: no rows in result set". We should have a better message saying "Block not found."

Scope/Requirements

The API handler would catch the not found error and return the new error message.

Urgency/Relative Priority

Quick win, we should just do it.

@winder winder added the FDE label Mar 1, 2021
@brianolson
Copy link
Contributor

HTTP 404 would seem appropriate

@mxmauro
Copy link

mxmauro commented May 10, 2021

Hi @winder , do you have an estimation when this will be implemented? Regards.

@winder winder added the good first issue Good for newcomers label May 10, 2021
@winder
Copy link
Contributor Author

winder commented May 10, 2021

@mxmauro unfortunately there's still a lot of other work ahead of this so we've been unable to get it into a sprint.

@mxmauro
Copy link

mxmauro commented May 10, 2021

Ok no problem. But it is important to know it will be changed in some future. Thanks for the feedback.

@algojack
Copy link
Contributor

algojack commented Oct 6, 2021

This is a problem for us since we have logs and alerts on metrics. And this is polluting the logs and setting off 5xx alarms. Noisy alarms lead to alarm fatigue, which leads to ignoring more important alarms. Please prioritize this @yaovi-a

@winder winder linked a pull request Oct 6, 2021 that will close this issue
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.

5 participants