Skip to content
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

signAndSend does not respect modified signedTransaction #11367

Open
valentunn opened this issue Mar 6, 2025 · 0 comments
Open

signAndSend does not respect modified signedTransaction #11367

valentunn opened this issue Mar 6, 2025 · 0 comments

Comments

@valentunn
Copy link
Contributor

Hey, thats Valentin from Nova Wallet

We are currently testing integration of wrapped proxy signing based on recently implemented allowCallDataAlteration
The allowCallDataAlteration itself works as expected and the payload validation logic is getting skipped. However, we noticed that an attempt to send proxy-wrapped tx is failing with "Invalid signature" response from the node
After some debugging and investigaiton we found out that despite signedTransaction is handled in signViaSigner in polkadot-js api library, polkadot-js apps itself doesnt use signAndSend which pipes signedTransaction into observeSubscribe but actually uses signAsync and send separetely, which causes signedTransaction to be lost in between since signViaSigner mutates only signature and not the modified tx itself, so send uses combination of original extrinsic and signature for the modified transaction

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant