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

Transaction.populate() Creates a message with accounts in unexpected order #23610

Closed
billyjacoby opened this issue Mar 11, 2022 · 2 comments
Closed

Comments

@billyjacoby
Copy link

billyjacoby commented Mar 11, 2022

Problem

When creating a transaction from a base58 encoded message using the Transaction.populate() function, the order of the accountKeys does not match the order of the accountKeys the original message. The account keys are all the same, but them being in a different order is causing issues.

This is leading to a signature verification failure, even though the two transactions technically have the same information in them.

I have created a repository that replicates this issue, and includes JSON outputs of the message before and after being passed in to the Transaction.populate() method.

The repo can be found here.

Proposed Solution

Figure out how to have the Transaction.populate() function return a message that has the accountKeys in the same order as the message it is constructed from.

@t-nelson
Copy link
Contributor

duplicate of #21722 (from calls populate)

@github-actions
Copy link
Contributor

This issue has been automatically locked since there has not been any activity in past 7 days after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 30, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants