Indexer log level spam#252
Open
MaryammAli wants to merge 3 commits into
Open
Conversation
Indexer log level spam
Indexer failure on Transfer to self
Contributor
|
resolve conflicts @MaryammAli |
Author
|
@dDevAhmed |
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
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.
Refine Indexer Log Level Spam
Overview
This PR refines indexer logging behavior by reducing unnecessary log noise generated during block processing.
Changes Made
"Processed Block"log messages from higher-severity logging to thedebuglevel.Problem Addressed
During audit review, excessive indexer logging was identified as a source of log spam, making operational monitoring more difficult and increasing unnecessary log verbosity.
This update improves log clarity while preserving debugging visibility when debug logging is enabled.
Acceptance Criteria
"Processed Block"logging now uses thedebuglog level.Notes
References
BE-206Labels
Fix Indexer Failure on Transfer to Self
Overview
This PR refines indexer transaction handling to properly support self-transfer scenarios where the sender and recipient addresses are identical (
from == to).Changes Made
Problem Addressed
An audit identified that the indexer could fail when processing transactions where the transfer source and destination were the same account.
This update ensures the indexer processes these transactions safely and consistently without triggering unexpected failures or invalid state transitions.
Acceptance Criteria
from == to) scenarios are handled correctly.Notes
References
BE-210Labels
Closes #206
Closes #210