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
Malformatted L1 Deposit Log #3
Comments
We added the |
Found the off by one: Here it increments the offset 6 times with 32 bytes to get to the dynamic remainder of the log data. It starts at 0, not at 32 though, so it should be Will fix, thanks for the bug report! |
Ok. I'm going to double check the contract version. Skimming the L1 contract, it seems like it hasn't been updated in over a week (other than some solc changes) |
FYI there are 7 fields, but I'm less familiar with how dynamic length fields / bools are packed in |
Example log split at 32 byte intervals
|
Updated |
The |
Ok. I got it. I used the new version and was able to see the balance of the account increase on L2 |
Setup
proto/bellatrix-api-fixes
origin/optimism-prototype
input-deriv-test-cleanup
Sending a deposit Tx in the following way (want to mint L2 eth, no contracts or data).
Result:
The text was updated successfully, but these errors were encountered: