From 924a0718b5076f08e5ad4614d07234292880c58f Mon Sep 17 00:00:00 2001
From: olivermrbl <oliver@mrbltech.com>
Date: Thu, 5 Sep 2024 11:01:56 +0200
Subject: [PATCH] cleanup

---
 .../auth-emailpass/src/services/emailpass.ts       | 14 +++-----------
 1 file changed, 3 insertions(+), 11 deletions(-)

diff --git a/packages/modules/providers/auth-emailpass/src/services/emailpass.ts b/packages/modules/providers/auth-emailpass/src/services/emailpass.ts
index 9316cdda6d728..ae060cda9c7d0 100644
--- a/packages/modules/providers/auth-emailpass/src/services/emailpass.ts
+++ b/packages/modules/providers/auth-emailpass/src/services/emailpass.ts
@@ -42,10 +42,10 @@ export class EmailPassAuthService extends AbstractAuthModuleProvider {
   }
 
   async update(
-    providerMetadata: Record<string, unknown>,
+    data: { email: string; password: string },
     authIdentityService: AuthIdentityProviderService
   ) {
-    const { email, password } = providerMetadata ?? {}
+    const { email, password } = data ?? {}
 
     if (!email || !isString(email)) {
       return {
@@ -54,8 +54,6 @@ export class EmailPassAuthService extends AbstractAuthModuleProvider {
       }
     }
 
-    // The only update allowed is the password
-    // Question: Should we throw if not provided?
     if (!password || !isString(password)) {
       return { success: true }
     }
@@ -74,15 +72,9 @@ export class EmailPassAuthService extends AbstractAuthModuleProvider {
       return { success: false, error: error.message }
     }
 
-    const copy = JSON.parse(JSON.stringify(authIdentity))
-    const providerIdentity = copy.provider_identities?.find(
-      (pi) => pi.provider === this.provider
-    )!
-    delete providerIdentity.provider_metadata?.password
-
     return {
       success: true,
-      authIdentity: copy,
+      authIdentity,
     }
   }