feat: add redirectUser callback (#136)

* redirectUser callback

* pass snapId

src/SnapKeyring.test.ts

@@ -28,6 +28,7 @@ describe('SnapKeyring', () => {
       await handleUserInput(true);
       return Promise.resolve();
     }),
+    redirectUser: jest.fn(async () => Promise.resolve()),
   };
 
   const snapId = 'local:snap.mock';
@@ -239,15 +240,10 @@ describe('SnapKeyring', () => {
     });
 
     it.each([
-      [
-        { message: 'Go to dapp to continue.' },
-        'The snap requested a redirect: message="Go to dapp to continue.", url=""',
-      ],
-      [
-        { url: 'https://example.com/sign?tx=1234' },
-        'The snap requested a redirect: message="", url="https://example.com/sign?tx=1234"',
-      ],
-    ])('returns a redirect %s', async (redirect, message) => {
+      [{ message: 'Go to dapp to continue.' }],
+      [{ url: 'https://example.com/sign?tx=1234' }],
+      [{ url: 'https://example.com/sign?tx=12345', message: 'Go to dapp.' }],
+    ])('returns a redirect %s', async (redirect) => {
       const spy = jest.spyOn(console, 'log').mockImplementation();
 
       mockSnapController.handleRequest.mockResolvedValue({
@@ -266,14 +262,24 @@ describe('SnapKeyring', () => {
         params: { id: requestId },
       });
 
+      const { url = '', message = '' } = redirect as {
+        url?: string;
+        message?: string;
+      };
+
       // We need to await on the request promise because the request submission
       // is async, so if we don't await, the test will exit before the promise
       // gets resolved.
       await expect(requestPromise).rejects.toThrow(
         'Request rejected by user or snap.',
       );
 
-      expect(console.log).toHaveBeenCalledWith(message);
+      // Check that `redirectUser` was called with the correct parameters
+      expect(mockCallbacks.redirectUser).toHaveBeenCalledWith(
+        snapId,
+        url,
+        message,
+      );
       spy.mockRestore();
     });
 

src/SnapKeyring.ts

@@ -72,6 +72,7 @@ export type SnapKeyringCallbacks = {
     snapId: string,
     handleUserInput: (accepted: boolean) => Promise<void>,
   ): Promise<void>;
+  redirectUser(snapId: string, url: string, message: string): Promise<void>;
 };
 
 /**
@@ -444,13 +445,10 @@ export class SnapKeyring extends EventEmitter {
       return response.result;
     }
 
-    // In the future, this should be handled by the UI. For now, we just log
-    // the redirect information for debugging purposes.
+    // If the snap answers asynchronously, we will inform the user with a redirect
     if (response.redirect?.message || response.redirect?.url) {
       const { message = '', url = '' } = response.redirect;
-      console.log(
-        `The snap requested a redirect: message="${message}", url="${url}"`,
-      );
+      await this.#callbacks.redirectUser(snapId, url, message);
     }
 
     return promise.promise;