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

writeReplayMesssage #55

Open
wilsoncusack opened this issue Sep 7, 2023 · 3 comments
Open

writeReplayMesssage #55

wilsoncusack opened this issue Sep 7, 2023 · 3 comments
Labels
alpha1 state: backlog This is pending further review type: enhancement New feature or request

Comments

@wilsoncusack
Copy link
Collaborator

This would be an L2 action. I think that this should take a transaction hash and a gas limit, we would

  1. (optionally) verify that the transaction had a FailedRelayedMessage event emitted
  2. fetch the calldata of the transaction
  3. call relayMessage on the L2 cross domain messenger with the calldata from the first, (maybe new min gas limit?

See notes on this and gas limit here https://community.optimism.io/docs/protocol/deposit-flow/#replays-in-action

@wilsoncusack
Copy link
Collaborator Author

actually I think this is relevant to both L1 and L2

@wilsoncusack
Copy link
Collaborator Author

OK so actually there's probably a subtask here that is estimateRelayMessageGas which would proxy to estimate gas from viem but set the caller to 0x0000000000000000000000000000000000000001, which is an address that will cause the function to revert if it is not relayed correctly.

@roninjin10
Copy link
Contributor

I don't believe this is relavent to l1. It was relavent prebedrock but now replaying is done via the same methods the happy path uses

@wbnns wbnns added state: backlog This is pending further review type: enhancement New feature or request labels Oct 4, 2023
@zencephalon zencephalon removed their assignment Oct 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alpha1 state: backlog This is pending further review type: enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants