RabbyHub / Rabby

The game-changing wallet for Ethereum and all EVM chains
https://rabby.io
Other
1.41k stars 408 forks source link

Submitting a transaction with Ledger results in error roughly ~30% of times for all of our users #2547

Closed 0xshippor closed 1 month ago

0xshippor commented 1 month ago

Hey everyone!

We have recently started our public testnet and were recommending users to use Rabby. We have distributed mock tokens to wallets of our users on testnet based on their specific token amounts on mainnet. Now a lot of those wallets, being used on mainnet, were hardware wallets namely Ledger. Our testnet runs on arbitrum sepolia.

Users have been reporting following error Incompatible EIP155-based V 843518 and chain id 421614. See the Common parameter of the Transaction constructor to set the chain id. after signing transaction on their Ledger.

rabby_error

Example raw transaction shown in Rabby while signing

{
  "chainId": 421614,
  "data": "0x8634d9c800000000000000000000000000000000000000000000000000000000000000200000000000000000000000000000000000000000000000000000000000000000",
  "from": "0x0c366680d1b8d01a194e90be1857f8186ae0e4bb",
  "gas": "0x2ac48",
  "gasPrice": "0x7270e00",
  "nonce": "0x13",
  "to": "0x4Ac6044247C569D7E5b5c7B609aADF3C63D75211",
  "value": "0x0"
}

(this is just our custom multicall with empty batch execution array)

This fails around 30% of the tries. Me and our users were not experiencing this issue on Metamask with Ledger connected so I assume the issue is on Rabby's side. If needed I can provide the code but it really is just any write function call on arbitrum sepolia and error is triggered sporadically.

Things I tried

Codesandbox code link Codesandbox app link

To reproduce

EDIT: Added simple example

Thank you for working on this awesome project, we and our community really love UX of Rabby.

vvvvvv1vvvvvv commented 1 month ago

@heisenberg-2077 can you help to check this?

heisenberg-2077 commented 1 month ago

Same issue https://github.com/LedgerHQ/ledger-live/issues/5265

vvvvvv1vvvvvv commented 1 month ago

going to close this issue since it's issue of Ledger, please report to Ledger team

lambertkevin commented 3 weeks ago

Problem should have been fixed by https://github.com/LedgerHQ/ledger-live/pull/8175 👍