-
Notifications
You must be signed in to change notification settings - Fork 1k
Description
We define in the SDK SigningTxData, which is a support type to sign transactions, as follows:
After #3883 and #3900 it seems like the fee_payer field is too limited. We don't always want to sign the wrapper transaction or sometimes we might have an offline signature that we'd want to use instead of producing a new one. This seems to lead to some workarounds in the codebase to allow these different workflows: we should think about reworking this piece of data to support these cases (no signature or load a signature).
Also, as a minor note, there are a couple of function (tx_signers and aux_signing_data) that specifically handle the MASP case: we should see if we can collapse the MASP case to some other case to avoid this specific handling (like not passing an owner when the source of a tx is the MASP)