Fix a memory leak and loading on amber login

src/app/components/LogIn.svelte

@@ -17,15 +17,17 @@
 
   const signUp = () => pushModal(SignUp)
 
-  const withLoading = (cb: () => void) => async () => {
+  const withLoading =
-    loading = true
+    (cb: (...args: any[]) => any) =>
+    async (...args: any[]) => {
+      loading = true
 
-    try {
+      try {
-      await cb()
+        await cb(...args)
-    } finally {
+      } finally {
-      loading = false
+        loading = false
+      }
     }
-  }
 
   const onSuccess = async (session: Session, relays: string[] = []) => {
     addSession(session)
@@ -50,7 +52,7 @@
     }
   })
 
-  const loginWithSigner = async (app: any) => {
+  const loginWithSigner = withLoading(async (app: any) => {
     const signer = new Nip55Signer(app.packageName)
     const pubkey = await signer.getPubkey()
 
@@ -62,7 +64,7 @@
         message: "Something went wrong! Please try again.",
       })
     }
-  }
+  })
 
   const loginWithBunker = () => pushModal(LogInBunker)
 

src/routes/+layout.svelte

@@ -209,7 +209,7 @@
         })
 
         // Listen for new notifications/memberships
-        subscribePersistent({
+        unsubRooms = subscribePersistent({
           relays,
           filters: [
             {kinds: [THREAD], since},