You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
On Friday, Feb 23, 2024, a partner notified the OP Labs team about a bug in the eth-optimism/sdk that would sometimes cause withdrawal proof submissions to revert. A fix has been released in eth-optimism/sdk v3.2.1 to resolve the bug.
We strongly recommend that teams using the eth-optimism SDK update their dependency with this version. We also encourage developers of alternative SDKs to incorporate similar changes to their code to avoid potential issues of their SDKs being unable to prove certain types of withdrawals.
As noted in the fix, new logic has been added to handle a case where the final proof element is less than 32 bytes and the element exists inside of a branch node. The current implementation of the on-chain MPT contract can't handle this natively, so we appended an extra proof element to handle it instead. Full details of the issue and fix can be found in PR 9663.
Big thanks to the BNB Chain team for alerting us to this issue. As an open-source project, we encourage others to leverage our codebase and appreciate it when developers in our ecosystem support each other by flagging issues and opportunities to make the protocol even stronger.
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
-
Hi everyone!
On Friday, Feb 23, 2024, a partner notified the OP Labs team about a bug in the eth-optimism/sdk that would sometimes cause withdrawal proof submissions to revert. A fix has been released in eth-optimism/sdk v3.2.1 to resolve the bug.
We strongly recommend that teams using the eth-optimism SDK update their dependency with this version. We also encourage developers of alternative SDKs to incorporate similar changes to their code to avoid potential issues of their SDKs being unable to prove certain types of withdrawals.
As noted in the fix, new logic has been added to handle a case where the final proof element is less than 32 bytes and the element exists inside of a branch node. The current implementation of the on-chain MPT contract can't handle this natively, so we appended an extra proof element to handle it instead. Full details of the issue and fix can be found in PR 9663.
Big thanks to the BNB Chain team for alerting us to this issue. As an open-source project, we encourage others to leverage our codebase and appreciate it when developers in our ecosystem support each other by flagging issues and opportunities to make the protocol even stronger.
Beta Was this translation helpful? Give feedback.
All reactions