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

Do call #11

Merged
merged 7 commits into from
Oct 21, 2020
Merged

Do call #11

merged 7 commits into from
Oct 21, 2020

Conversation

i-norden
Copy link
Collaborator

@i-norden i-norden commented Sep 25, 2020

Adding eth_Call endpoint to ipld-eth-server.

See #10

Included in this work are some Postgres functions for retrieving the canonical header at a provided height (based on weight).

@ramilexe
Copy link
Contributor

#10

@i-norden i-norden requested review from AFDudley and ramilexe October 7, 2020 14:01
@i-norden
Copy link
Collaborator Author

i-norden commented Oct 7, 2020

This is ostensibly completed, but I need to write unit tests. This is cumbersome because we need to generate sufficient state to perform the eth_Call operations on to test them.

@i-norden
Copy link
Collaborator Author

i-norden commented Oct 7, 2020

In addition to tests, #13 and #14 required to consider this complete

@i-norden
Copy link
Collaborator Author

i-norden commented Oct 9, 2020

vulcanize/ipld-eth-indexer#27 is blocking this, otherwise the tests are passing

pkg/eth/backend.go Show resolved Hide resolved
pkg/eth/backend.go Show resolved Hide resolved
pkg/eth/eth_call_test.go Show resolved Hide resolved
go.mod Outdated
)

replace github.com/ethereum/go-ethereum v1.9.11 => github.com/vulcanize/go-ethereum v1.9.11-statediff-0.0.5
replace github.com/ethereum/go-ethereum v1.9.11 => /Users/iannorden/go/src/github.com/ethereum/go-ethereum
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Full path doesn't work in CI

Copy link
Collaborator Author

@i-norden i-norden Oct 21, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah I had to wait for https://github.com/vulcanize/ipld-eth-indexer/releases/tag/v0.6.0-alpha, was just stashing on the remote, updated it now

@i-norden i-norden changed the title [WIP] Do call Do call Oct 21, 2020
@i-norden
Copy link
Collaborator Author

https://github.com/vulcanize/ipld-eth-indexer/releases/tag/v0.6.0-alpha is cut. Updated to use it and eth_call and rest of the tests are passing.

@i-norden i-norden merged commit ce706d0 into master Oct 21, 2020
@i-norden i-norden deleted the do_call branch October 27, 2020 19:21
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