safe-global / safe-wallet-web

Safe{Wallet} – smart contract wallet for Ethereum (ex-Gnosis Safe multisig)
https://app.safe.global
GNU General Public License v3.0
311 stars 360 forks source link

improve detection of safe multisend contract #3856

Open banteg opened 2 weeks ago

banteg commented 2 weeks ago

safe shows unexpected delegate call on some networks when authoring transactions with safe-eth-py and brownie-safe.

i found it boils down to safe-eth-py preferring to use either multisend deployment instead of the hardcoded one.

we can avoid this by instead hardcoding safe multisend contract bytecodes and simply allow delegate call into any address without ui warnings if its code matches this bytecode.