search

celo-org / op-geth

GNU Lesser General Public License v3.0
0 stars 0 forks source link

This is a branch for use with a migrated celo datadir #122

Closed piersy closed 3 weeks ago

piersy commented 1 month ago

This contains all changes required to get op-geth to start up and be able to process all the custom datatypes that existed in celo-blockchain during syncing. This doesn't contain any changes to support access over the RPC.

Signing via a types.Signer and transaction JSON Marshaling (which I believe is only used for tx signing operations), is supported only for CeloDynamicTxV2s (type 123). This code was pre-existing in the celo4 branch, we could further remove this code because it is not required for sync but it seems simpler to leave it as is.

Note we will need to implement types.Signers for legacy celo transactions since in order to return them over the rpc api since we will need to derive the senders.

Note we probably don't want to merge this right now as it will break a lot of tests since the RPC api will not work.

Fixes https://github.com/celo-org/celo-blockchain-planning/issues/256

piersy commented 3 weeks ago

Closing as I will be opening some more targeted PRs with this functionality