Merge pull request #710 from doitian/fix-new-tx-tracer-condition

quake · web-flow · commit 63b9412e1147 · 2025-05-20T18:55:58.000+09:00
fix: new tracer should have len 1
diff --git a/crates/fiber-lib/src/ckb/tx_tracing_actor.rs b/crates/fiber-lib/src/ckb/tx_tracing_actor.rs
@@ -172,7 +172,7 @@ impl CkbTxTracingState {
         let tx_tracers = self.tracers.entry(tx_hash).or_default();
         tx_tracers.tracers.push(tracer);
 
-        if tx_tracers.tracers.len() > 1 {
+        if tx_tracers.tracers.len() == 1 {
             // If there's no existing tracer for the tx, trigger the tracing task now
             self.run_tracers(myself, Some(vec![tx_hash])).await?
         }

Original file line number	Diff line number	Diff line change
`@@ -172,7 +172,7 @@ impl CkbTxTracingState {`
`172`	`172`	`let tx_tracers = self.tracers.entry(tx_hash).or_default();`
`173`	`173`	`tx_tracers.tracers.push(tracer);`
`174`	`174`
`175`		`- if tx_tracers.tracers.len() > 1 {`
	`175`	`+ if tx_tracers.tracers.len() == 1 {`
`176`	`176`	`// If there's no existing tracer for the tx, trigger the tracing task now`
`177`	`177`	`self.run_tracers(myself, Some(vec![tx_hash])).await?`
`178`	`178`	`}`