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

Rework the gas limit on Optimism Messengers #1938

Open
fredo opened this issue Jun 14, 2023 · 0 comments
Open

Rework the gas limit on Optimism Messengers #1938

fredo opened this issue Jun 14, 2023 · 0 comments

Comments

@fredo
Copy link
Contributor

fredo commented Jun 14, 2023

We have a constant value of 1M gas as a minGasLimit set in a single variable which both messengers, OptimismL1Messenger and OptimismL2Messenger derive from.

This is fine as long as you don't send a message back to Optimism. This happens in our local e2e tests. This way it is impossible to set the correct value as the message is a nested message send back to Optimism.

In general it would be nice to separate those two. In addition, we could analyze the correct values we need to send messages across.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant