Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New eth_getAddressesInBlock method #452

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

perama-v
Copy link
Contributor

@perama-v perama-v commented Jul 29, 2023

Description

Introduction of a method to standardise the extraction of a set of addresses that are involved in an Ethereum block.

Dependencies

This method involves use of the concept of an address Appearance, defined here:

Motivation

Documentation of the method allows different execution clients to implement the method. The method
itself is useful for obtaining knowledge that end-users can use (information about which transactions are relevant to their address). Multiple implementations allows detection of errors present in a single implementation through cross-client testing.

Background

The UnchainedIndex consists of "address appearances". It is a distributed index constructed by calling an Ethereum node using the trueblocks-core software (https://github.com/TrueBlocks/trueblocks-core). It finds addresses as they appear in each block and publishes the index. With this new method it could call a node to find addresses.

Changes made

  • A specification for what an "address appearance" is
  • An eth_ namespace method for fetching every "address appearance" in a single block
  • Test case

Test case generator

A test case generator has been written (https://github.com/perama-v/unchained-reader). It uses data from the UnchainedIndex (and therefore trueblocks-core by extension) to create the
test cases for individual blocks.

An alternative test generator can be made with the chifra blocks command built into trueblocks-core.

@perama-v perama-v marked this pull request as draft July 29, 2023 10:23
@perama-v perama-v marked this pull request as ready for review August 8, 2023 12:39
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.

None yet

1 participant