-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
add arbitrum error already known #12870
Conversation
I see you added a changeset file but it does not contain a tag. Please edit the text include at least one of the following tags:
|
Quality Gate passedIssues Measures |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did we get an answer from Tenderly if this was indeed an intended change?
Last I heard, they acknowledged they may have mis-categorized this under 400 instead of 200.
If they agree and fix their side, do we still want to go ahead with this fix?
One of the RPCs on Arbitrum Sepolia returns 400 status code on TransactionAlreadyKnown error, that breaks our error classification and causes MultiNode to log critical error due to contradictions in results (one of the RPCs returns success and TransactionAlreadyKnown is classified as Unknown)