Skip to content

Commit cdc1133

Browse files
authored
logger/json: implement OnFaultV2 and wire into hooks to include rData on faults
1 parent 182e08a commit cdc1133

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

eth/tracers/logger/logger_json.go

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -74,6 +74,7 @@ func NewJSONLogger(cfg *Config, writer io.Writer) *tracing.Hooks {
7474
OnExit: l.OnExit,
7575
OnOpcode: l.OnOpcode,
7676
OnFault: l.OnFault,
77+
OnFaultV2: l.OnFaultV2,
7778
}
7879
return l.hooks
7980
}
@@ -92,15 +93,19 @@ func NewJSONLoggerWithCallFrames(cfg *Config, writer io.Writer) *tracing.Hooks {
9293
OnExit: l.OnExit,
9394
OnOpcode: l.OnOpcode,
9495
OnFault: l.OnFault,
96+
OnFaultV2: l.OnFaultV2,
9597
}
9698
return l.hooks
9799
}
98100

99101
func (l *jsonLogger) OnFault(pc uint64, op byte, gas uint64, cost uint64, scope tracing.OpContext, depth int, err error) {
100-
// TODO: Add rData to this interface as well
101102
l.OnOpcode(pc, op, gas, cost, scope, nil, depth, err)
102103
}
103104

105+
func (l *jsonLogger) OnFaultV2(pc uint64, op byte, gas uint64, cost uint64, scope tracing.OpContext, rData []byte, depth int, err error) {
106+
l.OnOpcode(pc, op, gas, cost, scope, rData, depth, err)
107+
}
108+
104109
func (l *jsonLogger) OnOpcode(pc uint64, op byte, gas, cost uint64, scope tracing.OpContext, rData []byte, depth int, err error) {
105110
memory := scope.MemoryData()
106111
stack := scope.StackData()

0 commit comments

Comments
 (0)