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
Executing a SignMessageLib.signMessage tx built from the tx builder ui should work.
For SignMessageLib.signMessage the tx builder should always use operation delegatecall instead of call. The latter will always fail as the library function is referring to Safe contract storage that is only available when executing the tx via delegatecall.
Obtained result
The tx builder wrongfully sets operation call for SignMessageLib.signMessage invocations and there is no ui input to manually change it to delegatecall.
Screenshots
Shows operation call wrongfully set for SignMessageLib.signMessage:
Bug description
Environment
Steps to reproduce
0xd53cd0aB83D845Ac265BE939c57F53AD838012c9
signMessage
0x68616c6c6f63
Expected result
Executing a
SignMessageLib.signMessage
tx built from the tx builder ui should work.For
SignMessageLib.signMessage
the tx builder should always use operationdelegatecall
instead ofcall
. The latter will always fail as the library function is referring to Safe contract storage that is only available when executing the tx viadelegatecall
.Obtained result
The tx builder wrongfully sets operation
call
forSignMessageLib.signMessage
invocations and there is no ui input to manually change it todelegatecall
.Screenshots
Shows operation
call
wrongfully set forSignMessageLib.signMessage
:Moved from safe-global/safe-wallet-web#3406
The text was updated successfully, but these errors were encountered: