Skip to content

Sync functionality should include transaction data so speed ups can function properly. #2993

@rickycodes

Description

@rickycodes

Describe the bug
The issue is caused when a user approves a transaction in the browser extension and then uses the mobile app to speed up that transaction. The duplicate transaction does not include the input data.

To Reproduce
Step 1: On a desktop/laptop with Chrome visit https://bridge.axieinfinity.com/deposit and fill in the information for the deposit.
Step 2: Confirm in Metamask but modify the gas to be low so the transaction will not complete.
Step 3: Sync with your phone using the mobile QR code sync (Settings -> Advanced -> Sync with Mobile).
Step 4: Using Metamask mobile app speed up the transaction. The new transaction will be missing its input data.
Step 5: When the transaction completes it will send the funds directly to the contract due to the input data missing.

Expected behavior
When transactions are synced the input data should also be included so speed ups can function properly.

Smartphone (please complete the following information):
Prerequisites: A Metamask wallet, Metamask mobile app on Android
Computer used: Window 10 (21H1) with Chrome (91.0.4472.124), Metamask Chrome Extension v9.8.0, Android Metamask App v2.5.0

Metadata

Metadata

Assignees

Labels

Sev1-highAn issue that may have caused fund loss or access to wallet in the past & may still be ongoingtype-bugSomething isn't working

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions