Skip to content

feat: implement beacon deposit wallet address filtering - #46

Merged
ajit2903 merged 1 commit into
masterfrom
copilot/research-deposit-issues
Aug 3, 2026
Merged

feat: implement beacon deposit wallet address filtering#46
ajit2903 merged 1 commit into
masterfrom
copilot/research-deposit-issues

Conversation

Copilot AI commented Aug 3, 2026

Copy link
Copy Markdown

apps/explorer/config/prod/blockscout.exs (added in d4ec405) set a deposit_wallet_addresses config key that no code ever read, used a wrong relative path for config_helper.exs, and would crash app boot if ETHEREUM_JSONRPC_VARIANT=blockscout (since the matching indexer variant file doesn't exist). Deposits were silently not being restricted as intended.

Changes

  • Removed apps/explorer/config/prod/blockscout.exs — dead config, wrong path, potential boot crash vector
  • config/runtime.exs — added wallet_addresses to Indexer.Fetcher.Beacon.Deposit config, sourced from INDEXER_BEACON_DEPOSIT_FETCHER_WALLET_ADDRESSES (comma-separated; empty = no filter, preserving existing behaviour)
  • Indexer.Fetcher.Beacon.Deposit — added wallet_addresses to struct, populated in handle_continue, threaded into get_logs_with_deposits/5
  • Explorer.Chain.Beacon.Deposit.get_logs_with_deposits/5 — conditionally appends WHERE transaction.from_address_hash IN (^wallet_addresses) when the list is non-empty
INDEXER_BEACON_DEPOSIT_FETCHER_WALLET_ADDRESSES=0x06EE840642a33367ee59fCA237F270d5119d1356

@vercel

vercel Bot commented Aug 3, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
ajit2903-blockscout Ready Ready Preview Aug 3, 2026 8:22am
ajit2903-blockscout2 Building Building Preview Aug 3, 2026 8:22am
blockscout Ready Ready Preview Aug 3, 2026 8:22am
blockscout-a9xo Ready Ready Preview Aug 3, 2026 8:22am
blockscout-e3sk Ready Ready Preview Aug 3, 2026 8:22am
blockscout-g2h9 Ready Ready Preview Aug 3, 2026 8:22am
blockscout-yyq6 Building Building Preview Aug 3, 2026 8:22am

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