fix: Session id updates (#247)

* fix: remove `removeSessionId` command

* fix: stop requiring signature

* fix: change pairing flow

src/chrome/background/message-handler.ts

@@ -130,17 +130,6 @@ export const BackgroundMessageHandler =
           .mapErr(() => ({ reason: 'failedToFocusLedgerTab' }))
       }
 
-      case messageDiscriminator.removeSessionId: {
-        chromeLocalStore.getSingleItem('sessionRouter').andThen((data) => {
-          if (!data) {
-            return okAsync(undefined)
-          }
-
-          const { [message.sessionId]: _, ...rest } = data
-          return chromeLocalStore.setSingleItem('sessionRouter', rest)
-        })
-      }
-
       case messageDiscriminator.walletResponse: {
         const sessionId = message.data?.metadata?.sessionId
         const walletPublicKey = message.data?.metadata?.walletPublicKey

src/chrome/content-script/content-script.ts

@@ -62,12 +62,6 @@ const handleExtensionInteraction = async (
   extensionInteraction: ExtensionInteraction,
 ) => {
   switch (extensionInteraction.discriminator) {
-    case 'removeSessionId':
-      await sendMessage(
-        createMessage.removeSessionId(extensionInteraction.sessionId),
-      )
-      break
-
     case 'openPopup':
       await sendMessage(createMessage.openParingPopup())
       break

src/chrome/dapp/dapp-client.ts

@@ -64,7 +64,7 @@ export const ChromeDAppClient = (logger: AppLogger) => {
       if (message.interactionId) {
         logger.warn({ reason: 'UnrecognizedDappRequest', message })
         return onDappRequest(addOriginToWalletInteraction(message))
-      }      
+      }
     })
   }
 

src/chrome/messages/_types.ts

@@ -13,7 +13,6 @@ export const messageDiscriminator = {
   getExtensionOptions: 'getExtensionOptions',
   getSessionRouterData: 'getSessionRouterData',
   setSessionRouterData: 'setSessionRouterData',
-  removeSessionId: 'removeSessionId',
   dAppRequest: 'dAppRequest',
   closeLedgerTab: 'closeLedgerTab',
   focusLedgerTab: 'focusLedgerTab',

src/chrome/messages/create-message.ts

@@ -26,10 +26,6 @@ export const createMessage = {
     discriminator: messageDiscriminator.setSessionRouterData,
     data,
   }),
-  removeSessionId: (sessionId: string) => ({
-    discriminator: messageDiscriminator.removeSessionId,
-    sessionId,
-  }),
   extensionStatus: (isWalletLinked: boolean) => ({
     eventType: 'extensionStatus',
     isExtensionAvailable: true,

src/pairing/pairing.tsx

@@ -5,9 +5,8 @@ import { logger } from 'utils/logger'
 import { config, radixConnectConfig } from 'config'
 import { useConnectionsClient } from './state/connections'
 import { useConnectorOptions } from './state/options'
-import { Subscription, filter, map, tap, withLatestFrom } from 'rxjs'
+import { Subscription, combineLatest, filter, map, switchMap, tap } from 'rxjs'
 import { useNavigate } from 'react-router-dom'
-import { blakeHashBufferToHex } from 'crypto/blake2b'
 import { ed25519 } from '@noble/curves/ed25519'
 import { getLinkingSignatureMessage } from 'crypto/get-linking-message'
 
@@ -70,10 +69,11 @@ export const Pairing = () => {
       connectorClient.connected$
         .pipe(
           filter(Boolean),
-          withLatestFrom(hexConnectionPassword$),
-          withLatestFrom(linkClientInteraction$),
+          switchMap(() =>
+            combineLatest([hexConnectionPassword$, linkClientInteraction$]),
+          ),
         )
-        .subscribe(([[, password], interaction]) => {
+        .subscribe(([password, interaction]) => {
           connectionsClient
             .addOrUpdate(password, interaction)
             .map(() => connectorClient.disconnect())

src/pairing/state/connections.ts

@@ -5,6 +5,7 @@ import { Message } from 'chrome/messages/_types'
 import { chrome } from 'jest-chrome'
 import { err, errAsync } from 'neverthrow'
 import { useEffect, useState } from 'react'
+import { logger } from 'utils/logger'
 
 export type Connection = {
   walletName: string
@@ -84,12 +85,16 @@ const ConnectionsClient = (connections?: Connections | null) => {
   const addOrUpdate = (password: string, interaction: Message) => {
     const walletPublickey = interaction.publicKey
     const signature = interaction.signature
-    const message = getLinkingSignatureMessage(Buffer.from(password, 'hex'))
-    const validSignature = ed25519.verify(signature, message, walletPublickey)
+    if (signature) {
+      const message = getLinkingSignatureMessage(Buffer.from(password, 'hex'))
+      const validSignature = ed25519.verify(signature, message, walletPublickey)
 
-    if (!validSignature) {
-      return errAsync({ cause: 'Invalid Signature' } as Error)
+      if (!validSignature) {
+        logger.warn('Invalid Signature')
+        // return errAsync({ cause: 'Invalid Signature' } as Error)
+      }
     }
+
     if (connections && connections[walletPublickey]) {
       connections[walletPublickey] = {
         ...connections[walletPublickey],