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

feat: SuperchainETHWrapper contract #167

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

tremarkley
Copy link
Contributor

@tremarkley tremarkley commented Sep 20, 2024

Part of #135

This contract facilitates sending ETH across chains within the Superchain by wrapping ETH into SuperchainWETH, relaying the SuperchainWETH to the destination chain using L2ToL2CrossDomainMessenger, and then unwrapping it back to ETH and sending it to the specified recipient on the destination chain.

Copy link

cloudflare-workers-and-pages bot commented Sep 20, 2024

Deploying supersim with  Cloudflare Pages  Cloudflare Pages

Latest commit: 12e0c76
Status: ✅  Deploy successful!
Preview URL: https://3598cf14.supersim.pages.dev
Branch Preview URL: https://harry-superchain-weth-wrappe.supersim.pages.dev

View logs

@tremarkley
Copy link
Contributor Author

tremarkley commented Sep 20, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tremarkley and the rest of your teammates on Graphite Graphite

@tremarkley tremarkley force-pushed the harry/superchain_weth_wrapper branch 2 times, most recently from cd11ab8 to 2177e9d Compare September 23, 2024 19:38
@tremarkley tremarkley changed the base branch from main to 09-23-feat_deploy_proxies September 23, 2024 19:38
@tremarkley tremarkley force-pushed the harry/superchain_weth_wrapper branch 2 times, most recently from aed2920 to b46dabb Compare September 23, 2024 20:22
@tremarkley tremarkley force-pushed the harry/superchain_weth_wrapper branch 2 times, most recently from e62e3d1 to 4484eb9 Compare September 23, 2024 22:13
@tremarkley tremarkley force-pushed the harry/superchain_weth_wrapper branch 2 times, most recently from dc865d8 to 26bdc7e Compare September 23, 2024 22:25
@tremarkley tremarkley changed the base branch from 09-23-feat_deploy_proxies to main September 23, 2024 22:25
@tremarkley tremarkley force-pushed the harry/superchain_weth_wrapper branch 4 times, most recently from 3a57c36 to ca52ac7 Compare September 23, 2024 22:46
@tremarkley tremarkley changed the title feat: SuperchainWETHWrapper contract feat: SuperchainETHWrapper contract Sep 24, 2024
@zainbacchus
Copy link

OH LETS GOOOOOO

@tremarkley tremarkley force-pushed the harry/superchain_weth_wrapper branch 4 times, most recently from 1e9f92c to 09291d7 Compare September 25, 2024 02:51
@tremarkley tremarkley changed the title feat: SuperchainETHWrapper contract feat: SuperchainETHWrapper contract Sep 25, 2024
@tremarkley tremarkley force-pushed the harry/superchain_weth_wrapper branch 2 times, most recently from 8d24f0a to 3ae91ae Compare September 25, 2024 20:39
@tremarkley tremarkley marked this pull request as ready for review September 25, 2024 20:45
@tremarkley tremarkley requested a review from a team as a code owner September 25, 2024 20:45
@@ -119,16 +117,11 @@ func (r *L2ToL2MessageRelayer) Start(indexer *L2ToL2MessageIndexer, clients map[
}
// Pad gas by 33%.
paddedGas := (gasEstimate / 3) + gasEstimate
mu.Lock()
Copy link
Contributor Author

@tremarkley tremarkley Sep 25, 2024

Choose a reason for hiding this comment

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

this mutex lock does not work properly when multiple messages are sent in the same transaction. simplest way to fix this is to remove the lock and move the transactor within the case block

@tremarkley tremarkley force-pushed the harry/superchain_weth_wrapper branch 2 times, most recently from 316bbc6 to f269f4e Compare October 4, 2024 17:27
@tremarkley tremarkley force-pushed the harry/superchain_weth_wrapper branch from f269f4e to 12e0c76 Compare October 4, 2024 17:28
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