Content pfp
Content
@
https://opensea.io/collection/evm-6
0 reply
0 recast
2 reactions

stuffonfire pfp
stuffonfire
@stuffonfire.eth
if (Address.isContract(msg.sender)) { revert error() } considered harmful for account abstraction :( any suggestions?
1 reply
0 recast
1 reaction

maurelian  pfp
maurelian
@maurelian.eth
Why do you want it?
1 reply
0 recast
1 reaction

stuffonfire pfp
stuffonfire
@stuffonfire.eth
trying to replicate a manual flow from within a AA wallet implementation, this guard prevents it. think I found a workaround but I don’t understand why it’s there?
1 reply
0 recast
0 reaction

maurelian  pfp
maurelian
@maurelian.eth
Oh, so you’re trying to get around that guard?
1 reply
0 recast
0 reaction

stuffonfire pfp
stuffonfire
@stuffonfire.eth
Yea
1 reply
0 recast
0 reaction

maurelian  pfp
maurelian
@maurelian.eth
I don't think there's any way to do that. It's unfortunate they included that check, bad practice.
1 reply
0 recast
1 reaction

Matt Solomon pfp
Matt Solomon
@msolomon.eth
Unlikely that this is an option, but perhaps could selfdestruct your wallet and call the target method in the constructor when you redeploy, since that check looks for codesize != 0 and it's 0 during construction (OZ is removing that method in 0.5.0)
1 reply
0 recast
1 reaction