fix undefined usage

wireapp · Jun 21, 2024 · 3a727ea · 3a727ea
1 parent e1d4ccf
commit 3a727ea
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/libs/wire-subsystems/src/Wire/UserKeyStore.hs b/libs/wire-subsystems/src/Wire/UserKeyStore.hs
@@ -102,6 +102,9 @@ forEmailKey k f = foldKey (fmap Just . f) (const (pure Nothing)) k
 forPhoneKey :: (Applicative f) => UserKey -> (Phone -> f a) -> f (Maybe a)
 forPhoneKey k f = foldKey (const (pure Nothing)) (fmap Just . f) k
 
+fromEither :: Either Email Phone -> UserKey
+fromEither = either userEmailKey userPhoneKey
+
 data UserKeyStore m a where
   LookupKey :: UserKey -> UserKeyStore m (Maybe UserId)
   InsertKey :: UserId -> UserKey -> UserKeyStore m ()

diff --git a/services/brig/src/Brig/API/Public.hs b/services/brig/src/Brig/API/Public.hs
@@ -1065,8 +1065,7 @@ beginPasswordReset ::
   Handler r ()
 beginPasswordReset (Public.NewPasswordReset target) = do
   checkAllowlist target
-  let key = undefined target
-  (u, pair) <- lift (liftSem $ createPasswordResetCode key) !>> pwResetError
+  (u, pair) <- lift (liftSem $ createPasswordResetCode (fromEither target)) !>> pwResetError
   loc <- lift $ wrapClient $ API.lookupLocale u
   lift $ case target of
     Left email -> sendPasswordResetMail email pair loc