Skip to content

Commit d63825f

Browse files
authored
fix(keyring-eth-ledger-bridge): increase ledger iframe timeout (#314)
### What is the current state of things and why does it need to change? We get a lot of Sentry reports about a ledger iframe timeout error ### What is the solution your changes offer and how does it work? As a first mitigation, we will increase the timeout duration and then evaluate if we need to come up with a more robust solution. On a side track, we are also making [ledger transport initialization non-blocking](MetaMask/metamask-extension#34574). ### Are there any issues or other links reviewers should consult to understand this pull request better? Relates to: MetaMask/metamask-extension#33176
1 parent 0597b08 commit d63825f

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

packages/keyring-eth-ledger-bridge/src/ledger-iframe-bridge.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ import {
1414

1515
const LEDGER_IFRAME_ID = 'LEDGER-IFRAME';
1616

17-
const IFRAME_MESSAGE_TIMEOUT = 4000;
17+
const IFRAME_MESSAGE_TIMEOUT = 8000;
1818

1919
export enum IFrameMessageAction {
2020
LedgerIsIframeReady = 'ledger-is-iframe-ready',

0 commit comments

Comments
 (0)