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.

Sign up for GitHub

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

Jump to bottom

fix: events: put the _signed_ message through the pipeline. #10079

Merged
merged 1 commit into from
Jan 20, 2023
Merged

fix: events: put the _signed_ message through the pipeline. #10079

merged 1 commit into from
Jan 20, 2023

Commits on Jan 20, 2023

  1. fix: events: put the _signed_ message through the pipeline. 

    We were putting the unsigned/VM message through the pipeline.
The events index was storing the _unsigned_ message CID.

However, the Eth tx hash index maps signed Delegated-signature message
CIDs to transaction hashes, i.e. it uses the _signed_ message CID.

As a result, eth_getLogs and other log-related methods were
unable to resolve the transaction hash from the index properly, and
would end up returning 0x00..00 in the transactionHash field.
    @raulk
    raulk committed Jan 20, 2023
    Configuration menu
    Copy the full SHA
    2c56687 View commit details
    Browse the repository at this point in the history