Remove custom triggering of OnLog tracing hook

m-Peter · m-Peter · commit 75031c705544 · 2025-02-03T11:26:45.000+02:00
diff --git a/api/debug.go b/api/debug.go
@@ -238,7 +238,7 @@ func (d *DebugAPI) TraceCall(
 			}
 		}
 	}
-	res, err := view.DryCall(
+	_, err = view.DryCall(
 		from,
 		to,
 		tx.Data,
@@ -250,12 +250,6 @@ func (d *DebugAPI) TraceCall(
 		return nil, err
 	}
 
-	for _, log := range res.Logs {
-		if tracer != nil && tracer.OnLog != nil {
-			tracer.OnLog(log)
-		}
-	}
-
 	return tracer.GetResult()
 }
 
diff --git a/services/evm/executor.go b/services/evm/executor.go
@@ -106,11 +106,6 @@ func (s *BlockExecutor) Run(
 			res.GasConsumed,
 		)
 	}
-	for _, log := range res.Logs {
-		if tracer != nil && tracer.OnLog != nil {
-			tracer.OnLog(log)
-		}
-	}
 	l.Debug().Msg("transaction executed successfully")
 
 	return nil

Original file line number	Diff line number	Diff line change
`@@ -238,7 +238,7 @@ func (d *DebugAPI) TraceCall(`
`238`	`238`	`}`
`239`	`239`	`}`
`240`	`240`	`}`
`241`		`- res, err := view.DryCall(`
	`241`	`+ _, err = view.DryCall(`
`242`	`242`	`from,`
`243`	`243`	`to,`
`244`	`244`	`tx.Data,`
`@@ -250,12 +250,6 @@ func (d *DebugAPI) TraceCall(`
`250`	`250`	`return nil, err`
`251`	`251`	`}`
`252`	`252`
`253`		`- for _, log := range res.Logs {`
`254`		`- if tracer != nil && tracer.OnLog != nil {`
`255`		`- tracer.OnLog(log)`
`256`		`- }`
`257`		`- }`
`258`		`-`
`259`	`253`	`return tracer.GetResult()`
`260`	`254`	`}`
`261`	`255`
Original file line number	Diff line number	Diff line change
`@@ -106,11 +106,6 @@ func (s *BlockExecutor) Run(`
`106`	`106`	`res.GasConsumed,`
`107`	`107`	`)`
`108`	`108`	`}`
`109`		`- for _, log := range res.Logs {`
`110`		`- if tracer != nil && tracer.OnLog != nil {`
`111`		`- tracer.OnLog(log)`
`112`		`- }`
`113`		`- }`
`114`	`109`	`l.Debug().Msg("transaction executed successfully")`
`115`	`110`
`116`	`111`	`return nil`