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

Deployment bug while L1->L2 erc20 bridging #14610

Closed
adaki2004 opened this issue Aug 30, 2023 · 3 comments
Closed

Deployment bug while L1->L2 erc20 bridging #14610

adaki2004 opened this issue Aug 30, 2023 · 3 comments

Comments

@adaki2004
Copy link
Contributor

Describe the bug

Jeff reported that the invokeMessageCall() always fails during L1 to L2 ERC20 bridging.(The other way it works !!).

I recalled that we faced this issue during testing so we mocked the Bridge and InvokeMessageCall() in a way of directly calling the encoded function (in message.data).

@dantaik Do you remember this convo ?

kép

It seems it might not be a forge-foundry issue but how chain handles the deployments with the new keyword.

As in A3, the deploymet was with Create2Upgradeable I'd go back to this:

kép

Steps to reproduce

Steps to reproduce here.

Additional context

Additional context here.

@cyberhorsey
Copy link
Contributor

I am unsure if this is the issue or not, but I will try a deployment of the previous deploy erc20 code and see if it works.

@cyberhorsey
Copy link
Contributor

Oh and note this same issue reappears on retryMessage, and whether it's claimed by relayer or by the bridge UI.

@davidtaikocha
Copy link
Member

Closed by #14621

@github-project-automation github-project-automation bot moved this from 📝 Todo to ✅ Done in Taiko Project Board Sep 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

No branches or pull requests

3 participants