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
fix: ensure failed CREATE2
action is finalized w/ proper frame
#10366
Conversation
Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>
Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>
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.
LGTM
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## develop #10366 +/- ##
=============================================
- Coverage 63.52% 63.51% -0.01%
- Complexity 30990 31008 +18
=============================================
Files 3341 3342 +1
Lines 134577 134698 +121
Branches 13994 14005 +11
=============================================
+ Hits 85484 85556 +72
- Misses 45736 45783 +47
- Partials 3357 3359 +2 ☔ View full report in Codecov by Sentry. |
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.
LGTM
Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>
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.
Makes sense. LGTM.
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.
LGTM !
Description:
CREATE2
needstraceAccountCreationResult()
tracer hook implemented in mono-service #10360tracePostExecution()
is never called upon aCREATE2
address collision, re-implementstraceAccountCreationResult()
in mono-serviceHederaTracer
and updates theMessageCallProcessors
not to make duplicate calls totracePostExecution()
.