Skip to content

Improve error handling #3

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
Closed
5 tasks done
PJColombo opened this issue Mar 6, 2023 · 0 comments · Fixed by #21
Closed
5 tasks done

Improve error handling #3

PJColombo opened this issue Mar 6, 2023 · 0 comments · Fixed by #21
Assignees
@PJColombo
Labels
enhancement New feature or request

Comments

@PJColombo
Copy link
Member

PJColombo commented Mar 6, 2023 •
edited
Loading

  • Handle slot processing errors via exponential backoff. Check out backoff crate
  • Use anyhow to easily bubble up/downcast errors.
  • Create custom errors for the beacon client by using the thiserror crate
  • Propagate slot processor's errors
  • Remove unwrap() calls
@PJColombo PJColombo transferred this issue from Blobscan/blobscan-indexer Mar 19, 2023
@PJColombo PJColombo changed the title Make indexer robust: handle errors properly Implement exponential backoff Mar 19, 2023
@PJColombo PJColombo changed the title Implement exponential backoff Improve error handling Mar 24, 2023
@PJColombo PJColombo mentioned this issue Mar 24, 2023
Merged
@PJColombo PJColombo self-assigned this Mar 24, 2023
@PJColombo PJColombo added the enhancement New feature or request label Mar 24, 2023
@PJColombo PJColombo linked a pull request Mar 28, 2023 that will close this issue
Error handling improvements #21
Merged
@0xGabi 0xGabi closed this as completed in #21 Apr 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@PJColombo PJColombo

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Error handling improvements Blobscan/blobscan-indexer.rs
1 participant
@PJColombo