[Sentry] Error: No keyring found for the requested account. Error info: There are no keyrings

[sentry-io]

Sentry Issue: METAMASK-WQNC

Error: No keyring found for the requested account. Error info: There are no keyrings
  at MetamaskController.getAccountType (app/scripts/metamask-controller.js:2491:5)
  at TransactionController._buildEventFragmentProperties (app/scripts/controllers/transactions/index.js:2460:7)
  at TransactionController._trackTransactionMetricsEvent (app/scripts/controllers/transactions/index.js:2645:25)

[gauthierpetetin]

Events in past 14 days: 306

[gauthierpetetin]

When a user tries to send a transaction or sign a message, MetaMask fetches the private key for the account from its keyring. This error means that no such keyring exists for the account in question.

[danjm]

Some relevant background in these threads:
https://consensys.slack.com/archives/GTQAGKY5V/p1650312797440969
https://consensys.slack.com/archives/GTQAGKY5V/p1633992646258900

[danjm]

This is a duplicate of #16193, but there is a great repro shared on that thread

[danjm]

Also see #14652

[sentry-io]

Sentry Issue: METAMASK-XB5P

[seaona]

Just encountered this issue in 12.3.0 version, find repro steps below

no-keyrings.mp4

Repro:

1. Connect to the test dapp
2. Trigger a tx
3. Disable the Extension right away
4. Re-enable the Extension
5. See error

[gauthierpetetin]

This PR shall mitigate this issue (at least partially): #29599

[mikesposito]

This error is thrown if almost any KeyringController method is called while the wallet is locked: https://github.com/MetaMask/core/blob/af90e6780c66cfdc734d6482c87a597c4c4deb36/packages/keyring-controller/src/KeyringController.ts#L942

This seems to be compatible with the reproduction steps shared in this issue and in this one

[desi]

MetaMask/core#5171 will give us clarity around this issue so we will wait to see the results of that work to return to this one.