Skip to content

feat: log all response error messages#852

Merged
Theodus merged 1 commit intomainfrom
release
Jun 19, 2024
Merged

feat: log all response error messages#852
Theodus merged 1 commit intomainfrom
release

Conversation

@Theodus
Copy link
Member

@Theodus Theodus commented Jun 19, 2024

This makes sure we log all error responses back to the client, even if it doesn't fall into the main client query handler. Not sure if there's a cleaner way to do this.

@Theodus Theodus requested a review from LNSD June 19, 2024 13:57
Copy link
Contributor

@LNSD LNSD left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM ✅

@Theodus Theodus merged commit 8c4c3e0 into main Jun 19, 2024
@Theodus Theodus deleted the release branch June 19, 2024 15:48
@Theodus Theodus mentioned this pull request Jun 20, 2024
Theodus added a commit that referenced this pull request Jun 20, 2024
# Release Notes
- fix: indexing progress cache, isolate by URL
- fix: remove excess timeout (#851)
- feat: log all response error messages (#852)
- fix: success rate cutoff at fixed time behind chain head (#853, #854)
- fix: select deployment outside of indexer-selection (#857)
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

Successfully merging this pull request may close these issues.

2 participants