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.

Sign up for GitHub

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

Jump to bottom

Unsuccessful Transactions Due to Bad Signature and Trap Error #243

Closed
Juanma0x opened this issue Aug 9, 2024 · 4 comments · Fixed by #244
Closed

Unsuccessful Transactions Due to Bad Signature and Trap Error #243

Juanma0x opened this issue Aug 9, 2024 · 4 comments · Fixed by #244

Comments

@Juanma0x
Copy link

Juanma0x commented Aug 9, 2024

When connected to Polkadot Wallet Snap Example dApp and attempting a simple transaction, I encountered the following error in the console:

MetaMask - RPC Error: 4003: Client error: Execution failed: Execution aborted due to trap: wasm trap: wasm unreachable instruction executed

When I use the Staking Dashboard, the operation is canceled, and I receive the following error in the console:

ExtrinsicStatus:: 1010: Invalid Transaction: Transaction has a bad signature

Perhaps it is caused by the runtime upgrade to v1.2.5?

Chrome 127.0.6533.100
Metamask 12.0.1
ChainSafe Snap 0.11.0

Note: PolkaGate snap does work.
@danforbes
Copy link
Collaborator

Thank you for reporting this issue, @Juanma0x 🙏🏻 We will look into it as soon as possible.

@ChainSafe ChainSafe deleted a comment Aug 12, 2024
@Lykhoyda Lykhoyda mentioned this issue Aug 21, 2024
Merged
@danforbes
Copy link
Collaborator

@Juanma0x - @Lykhoyda has opened a Pull Request to address this Issue.

Lykhoyda added a commit that referenced this issue Aug 27, 2024
@Lykhoyda
fix: wasm error during transaction (#244)
c2f51e5 
Closes #243 
- polkadot libraries were updated
- updates for Metamask cli and build 
- switched to node 18 (16 got deprecated with the new metamask cli)
- GitHub actions uses v 18 of node 
- Removed shasum check on GitHub actions as a new CLI updates the
manifest file if it doesn't match
@danforbes
Copy link
Collaborator

@Juanma0x - the new version of the Snap has been released and should be available from MetaMask. Is everything working on your end?

@michalisFr
Copy link

@Juanma0x - the new version of the Snap has been released and should be available from MetaMask. Is everything working on your end?

Hi @danforbes! Juanma was relaying an issue from a user. The user updated to the latest version and they report the issue is resolved. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: wasm error during transaction ChainSafe/metamask-snap-polkadot
3 participants
@danforbes @michalisFr @Juanma0x