search

MetaMask / snaps

Extend the functionality of MetaMask using Snaps
https://metamask.io/snaps/
Other
724 stars 557 forks source link

Suspend Snaps request processing timeout while waiting for a response from MetaMask #581

Closed rekmarks closed 2 years ago

rekmarks commented 2 years ago

Following #444, we should suspend the Snaps request processing timeout while the Snap is waiting for a response from a snaps_-prefixed method. A Snap may make requests for eth_ and other methods via wallet.request(), and we should not suspend the timeout for those. This feature should be implemented by means of some middleware that inspects JSON-RPC made from Snaps to MetaMask. Ideally, this should not require modifying the extension's own RPC pipeline, but should be encapsulated in the Snaps execution service and/or SnapController.

rekmarks commented 2 years ago

Hey team! Please add your planning poker estimate with ZenHub @david0xd @FrederikBolding @GuillaumeRx @hmalik88 @Mrtenz @ritave