-
Notifications
You must be signed in to change notification settings - Fork 464
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
Feature to allow decoding of call data for contracts that are proxies/diamonds #2657
Comments
Thanks for the proposal! Sounds like there's a real pain point for some users. I'll bring this up for discussion in the team. |
Hi guys! I'm Andrei, Safe{Wallet} PM. Thanks for pointing out this one and providing the proposed solution! No surprise but we got limited dev resource and 2023 schedule fully packed with features. There can be potentially 2 paths for this one to speed things up:
Hope that works for you! |
hey @kirkkonen appreciate the response. Its not just ERC-2535, its any Transparent, UUPS, Upgradeable, or DelegateCall Proxy, basically any form of proxy because the proxy's ABI doesn't contain the functions that are being called. They all suffer the same fate, which is the Safe UI cannot display the decoded function name and parameters. If you have a look at the issues list on https://github.com/safe-global/safe-transaction-service/issues there's many people requesting to be able to upload the abi for there proxies implementation. The resounding response is that Safe doesn't support proxies. I've written some custom internal tooling that solves this for us using https://openchain.xyz/signatures and wont have time in the short to medium term to open a PR but I will definitely look into it when time allows. |
What is needed?
For Users that use proxy contracts or diamonds ERC-2535 the SAFE UI does not decode the transaction data. This is a known issue. Users are also not able to upload the abi for these contracts in anyway.
I first submitted this over at safe-global/safe-transaction-service#1709
Feature:
This has been a problem for years for many users. Currently we must use a tool like https://lab.miguelmota.com/ethereum-input-data-decoder/example/ to verify call data or other custom tooling
Alternative:
Add a button that uses the https://www.4byte.directory/ or https://openchain.xyz/signatures api to check if there is a result for the function signature and if found use it to decode the call data.
Background
URL
https://app.safe.global/transactions/
The text was updated successfully, but these errors were encountered: