feat: enhance error logging #2055
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Added detailed error logging for relay errors, including status code and provider trailer.
nimrod-teich
approved these changes
Aug 20, 2025
Tomelia1999
added a commit
that referenced
this pull request
Aug 25, 2025
commit 8aebcdf Author: avitenzer <tenzer@clara.co.uk> Date: Wed Aug 20 16:20:42 2025 +0300 feat: enhance error logging (#2055) * enhance error logging in ResultsManager for better troubleshooting - Added detailed error logging for relay errors, including status code and provider trailer. * fixed an issue in the results manager * updated e2e error codes * solved issue in the tests * updated the allowedErrorList commit b151e5a Author: Yaroms <103432884+Yaroms@users.noreply.github.com> Date: Wed Aug 20 11:18:27 2025 +0300 feat: Parralel relay quorum (#2050) * feat: allow multiple relays on launch * support sending in parallel * pr changes * add header * adding sorting to json files * addin sorted json for comparing * add flags * add params struct * making the quorum based on unique results and not just results amount * return user error if the headers are wrong * input checks * fix unitest if failed to canonicaljson, print error instead of skipping * fix tests with default values * allow canonical errors --------- Co-authored-by: Yarom Swisa <yarom@lavanet.xyz git config --global user.name Yarom> Co-authored-by: Yaroms <yaroms@lavanet.xyz> Co-authored-by: avitenzer <tenzer@clara.co.uk> commit 377114f Merge: 0b9ec6f fb59765 Author: Yaroms <103432884+Yaroms@users.noreply.github.com> Date: Tue Aug 12 14:31:36 2025 +0300 Merge pull request #2054 from lavanet/hedera-spec feat: add hedera spec commit fb59765 Author: Yarom Swisa <yarom@lavanet.xyz git config --global user.name Yarom> Date: Mon Aug 11 15:26:39 2025 +0300 feat: add hedera spec
nimrod-teich
added a commit
that referenced
this pull request
Sep 2, 2025
* git ignore * backup providers scripts * adding backup-providers flag to config file * tests and mock * adding backup provider logic to csm * adding comments to provider_optimizer file * adding comments to WeightedChoice * update backup_provider * unify functions * Squashed commit of the following: commit 8aebcdf Author: avitenzer <tenzer@clara.co.uk> Date: Wed Aug 20 16:20:42 2025 +0300 feat: enhance error logging (#2055) * enhance error logging in ResultsManager for better troubleshooting - Added detailed error logging for relay errors, including status code and provider trailer. * fixed an issue in the results manager * updated e2e error codes * solved issue in the tests * updated the allowedErrorList commit b151e5a Author: Yaroms <103432884+Yaroms@users.noreply.github.com> Date: Wed Aug 20 11:18:27 2025 +0300 feat: Parralel relay quorum (#2050) * feat: allow multiple relays on launch * support sending in parallel * pr changes * add header * adding sorting to json files * addin sorted json for comparing * add flags * add params struct * making the quorum based on unique results and not just results amount * return user error if the headers are wrong * input checks * fix unitest if failed to canonicaljson, print error instead of skipping * fix tests with default values * allow canonical errors --------- Co-authored-by: Yarom Swisa <yarom@lavanet.xyz git config --global user.name Yarom> Co-authored-by: Yaroms <yaroms@lavanet.xyz> Co-authored-by: avitenzer <tenzer@clara.co.uk> commit 377114f Merge: 0b9ec6f fb59765 Author: Yaroms <103432884+Yaroms@users.noreply.github.com> Date: Tue Aug 12 14:31:36 2025 +0300 Merge pull request #2054 from lavanet/hedera-spec feat: add hedera spec commit fb59765 Author: Yarom Swisa <yarom@lavanet.xyz git config --global user.name Yarom> Date: Mon Aug 11 15:26:39 2025 +0300 feat: add hedera spec * lint * remove cu vlidation for backup providers * adding validations to backup providers' chain-id * Squashed commit of the following: commit 9335922 Author: AnnaR-prog <anna@magmadevs.com> Date: Wed Aug 27 17:24:15 2025 +0300 fix: The following has been fixed in the quorum feature (#2057) * [bug fixes]: The following has been fixed in the quorum feature: 1. Retry condition when the quorum is not met 2. Calculation of actualQuorumNeeded after every Retry 3. Error represented to the user after quorum is not met * fix: changes after CR * unify 2 IFs into 1 * revert IFs unify * unify 2 IFs into 1 --------- Co-authored-by: Nim Rod <nimrod.teich@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Closes: #XXXX
Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...
!in the type prefix if API or client breaking changemainbranchReviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...