Skip to content

fix(txpool): use instance logger instead of global log#18797

Merged
taratorio merged 1 commit into
erigontech:mainfrom
Bilogweb3:fix/txpool-logger-instance
Jan 26, 2026
Merged

fix(txpool): use instance logger instead of global log#18797
taratorio merged 1 commit into
erigontech:mainfrom
Bilogweb3:fix/txpool-logger-instance

Conversation

@Bilogweb3
Copy link
Copy Markdown
Contributor

Fixed 4 places in pool.go where log.Debug was used instead of p.logger.Debug:

  • Line 1059: blob validation error
  • Line 1068: Osaka proof validation error
  • Line 2091: NoGossip drain message
  • Line 2178: NoGossip peer sync message

This was inconsistent with the rest of the file which uses p.logger throughout. Using global log bypasses any logger configuration set on the pool instance.

@taratorio taratorio enabled auto-merge (squash) January 26, 2026 01:10
@taratorio taratorio merged commit e605c98 into erigontech:main Jan 26, 2026
15 of 16 checks passed
mattevans added a commit to ethpandaops/erigone-legacy that referenced this pull request Jan 28, 2026
* main: (37 commits)
  Make `lastCommittedTxNum` atomic (erigontech#18820)
  integrity: fix redundant variable name in checkLogIdx (erigontech#18646)
  Fixed index building for v0 snapshot format (erigontech#18825)
  execution/vm: fold EVMInterpreter into EVM (erigontech#18765)
  fix: return correct prune mode value instead of nil (erigontech#18804)
  fix BuildMissedAccessors for caplin state (erigontech#18802)
  execution: EIP-7708: ETH transfers emit a log (erigontech#18761)
  refactor(rpc): remove redundant boolean comparison in AccountRange (erigontech#18814)
  Show the default P2P discovery bools in --help (erigontech#18818)
  fix(txpool): remove redundant Worst() calls in queued pool discard loop (erigontech#18810)
  node: remove unused logger and test helper (erigontech#18744)
  Disabled compression in collate function (erigontech#18813)
  Add missing tx.Commit() in stage_snapshots (erigontech#18816)
  Bumped major version of snapshot files (erigontech#18815)
  qa-tests: summary for RPC integration tests (erigontech#18809)
  perf: reuse hash buffer in witness storage leaf hash (erigontech#18803)
  execution/stagedsync: remove canPrune from serial_exec (erigontech#18800)
  Fixed deduplication intervals for rebuild cmd (erigontech#18796)
  Fix race due to warmup cache (erigontech#18799)
  fix(txpool): use instance logger instead of global log (erigontech#18797)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants