Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

[web3.js] Discrepancy between legacy Transaction API and new VersionedTransaction API #28900

Closed
vovacodes opened this issue Nov 20, 2022 · 2 comments · Fixed by #28990
Closed

[web3.js] Discrepancy between legacy Transaction API and new VersionedTransaction API #28900

vovacodes opened this issue Nov 20, 2022 · 2 comments · Fixed by #28990
Assignees
@steveluscher
Labels
community Community contribution javascript Pull requests that update Javascript code
Milestone
web3.js Roadmap –...

Comments

@vovacodes
Copy link
Contributor

vovacodes commented Nov 20, 2022 •
edited by steveluscher
Loading

Problem

Version: @solana/web3.js@1.66.2

Combination of VersionedTransaction.deserialize and TransactionMessage.decompile produces different result from the legacy Transaction.from API when deserializing the same exact transaction.

Reproduction below (note the difference in the isWritable flag of the last key)
Screenshot 2022-11-20 at 23 31 41

https://codesandbox.io/p/sandbox/funny-ride-1zmikd

While not 100% sure, I believe the legacy API produces the correct result and the new API is being wrong.

Serialized transaction 
AQAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAABAAUHLl06XmBMUgEaGqFEDWXlrfuG9BxuqyiqilWnn+lBhO3ljbFniLvsim6dTEc5cyoTN2b5SA3mE/vy52heCKtBsAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAACK/25BBZJGavm0hr5XZ58vaLQc3cMeAgkndKj2Ni7RPDGxjMPhSKClKTgYkg7frtqzkeSTNsC3TbZp0QRwNCSwUhn4magdT/hPtZPS7fipCsGzqzQlj33yM+pQMCsb0uBX82VZkozhurrbZbi12eMbon+FOb7GAsyxrcKvufRnDqK6IjeTZQF0WwCoTTZmOO6HR7bBk26zWjZL9jPKBtpQEFBgAGAQMEAhHyI8aJUuHytv5AQg8AAAAAAA==
@vovacodes vovacodes added the community Community contribution label Nov 20, 2022
@steveluscher steveluscher added the javascript Pull requests that update Javascript code label Nov 21, 2022
@steveluscher
Copy link
Contributor

Thanks for this! Super helpful. I'll take a look in a bit.

@steveluscher steveluscher self-assigned this Nov 21, 2022
@steveluscher
Copy link
Contributor

Found the bug. I'm so grateful for your find!

@steveluscher steveluscher mentioned this issue Nov 30, 2022
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@steveluscher steveluscher

Labels
community Community contribution javascript Pull requests that update Javascript code
Projects
None yet
Milestone
web3.js Roadmap – November 2022
Development

Successfully merging a pull request may close this issue.

fix: TransactionMessage.decompile() now counts the correct number of unsigned, writable accounts
2 participants
@steveluscher @vovacodes