Skip to content

fix(session): preserve halved log scan ranges#462

Open
EfeDurmaz16 wants to merge 1 commit into
tempoxyz:mainfrom
EfeDurmaz16:fix/log-scan-range-halving
Open

fix(session): preserve halved log scan ranges#462
EfeDurmaz16 wants to merge 1 commit into
tempoxyz:mainfrom
EfeDurmaz16:fix/log-scan-range-halving

Conversation

@EfeDurmaz16
Copy link
Copy Markdown

Summary

  • keep both halves of an oversized eth_getLogs block range when the RPC asks for a smaller query
  • process pending split ranges before advancing below the original chunk, so the upper half is not silently skipped
  • add unit coverage for the split range boundaries

Closes #412.

Verification

  • PATH="$HOME/.cargo/bin:$PATH" cargo +nightly fmt --all -- --check
  • RUSTC="$(rustup which --toolchain stable rustc)" rustup run stable cargo test -p tempo-common payment::session::channel::tests
  • RUSTC="$(rustup which --toolchain stable rustc)" rustup run stable cargo test -p tempo-common
  • RUSTC="$(rustup which --toolchain nightly rustc)" PATH="$HOME/.cargo/bin:$PATH" cargo +nightly clippy -p tempo-common --all-targets --all-features --locked -- -D warnings

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.

bug: channel log scan drops blocks when RPC triggers range halving

1 participant